Re: [ptxdist] [PATCH RESEND 1/2] Add host-bison package

[Michael Olbrich <m.olbrich@pengutronix.de>]

Hi,

On Mon, Nov 07, 2011 at 09:07:02PM +0100, Bernhard Walle wrote:
> From: Bernhard Walle <walle@corscience.de>
> 
> Since we have host-flex, it makes also sense to get independent of
> the bison installation of the host.

I disagree with this. We have host-flex because on some older distributions
the installed flex break building some packages.
If we really want to remove the dependency on host system flex and yacc
(and I'm not sure that's a good idea), then we need to fix all packages.

I did a quick test (link yacc/bison, flex/lex to /bin/false in the PATH)
to see what fails:
at, bash, binutils, bluez, fbset, flex, gstreamer, host-bison, iproute2,
kbd, libgmp, libical, libpcap, memedit, xorg-server
This may not be complete. At least the configure check for yacc won't fail
this way, only trying to use it.

Note host-bison in the list. configure checks for lex and yacc. I think the
generated files are all shipped but the check is still there. :-(

Michael

> Signed-off-by: Bernhard Walle <walle@corscience.de>
> ---
>  rules/host-bison.in   |    8 ++++++++
>  rules/host-bison.make |   32 ++++++++++++++++++++++++++++++++
>  rules/pre/bison.make  |    5 +++++
>  3 files changed, 45 insertions(+), 0 deletions(-)
>  create mode 100644 rules/host-bison.in
>  create mode 100644 rules/host-bison.make
>  create mode 100644 rules/pre/bison.make
> 
> diff --git a/rules/host-bison.in b/rules/host-bison.in
> new file mode 100644
> index 0000000..41bb930
> --- /dev/null
> +++ b/rules/host-bison.in
> @@ -0,0 +1,8 @@
> +## SECTION=hosttools_noprompt
> +
> +config HOST_BISON
> +	tristate
> +	help
> +	  The parser generator (replaces yacc).
> +
> +# vim: syntax=kconfig
> diff --git a/rules/host-bison.make b/rules/host-bison.make
> new file mode 100644
> index 0000000..d62b0b4
> --- /dev/null
> +++ b/rules/host-bison.make
> @@ -0,0 +1,32 @@
> +# -*-makefile-*-
> +#
> +# Copyright (C) 2011 by Bernhard Walle <walle@corscience.de>
> +#
> +# See CREDITS for details about who has contributed to this project.
> +#
> +# For further information about the PTXdist project and license conditions
> +# see the README file.
> +#
> +
> +#
> +# We provide this package
> +#
> +HOST_PACKAGES-$(PTXCONF_HOST_BISON) += host-bison
> +
> +#
> +# Paths and names
> +#
> +HOST_BISON_VERSION	:= 2.5
> +HOST_BISON_MD5		:= 9dba20116b13fc61a0846b0058fbe004
> +HOST_BISON		:= bison-$(HOST_BISON_VERSION)
> +HOST_BISON_SUFFIX	:= tar.bz2
> +HOST_BISON_URL		:= http://ftp.gnu.org/gnu/bison/$(HOST_BISON).$(HOST_BISON_SUFFIX)
> +HOST_BISON_SOURCE	:= $(SRCDIR)/$(HOST_BISON).$(HOST_BISON_SUFFIX)
> +HOST_BISON_DIR		:= $(HOST_BUILDDIR)/$(HOST_BISON)
> +
> +#
> +# autoconf
> +#
> +HOST_BISON_CONF_TOOL	:= autoconf
> +
> +# vim: syntax=make
> diff --git a/rules/pre/bison.make b/rules/pre/bison.make
> new file mode 100644
> index 0000000..cfbee38
> --- /dev/null
> +++ b/rules/pre/bison.make
> @@ -0,0 +1,5 @@
> +# -*-makefile-*-
> +
> +ifdef PTXCONF_HOST_BISON
> +export BISON_PKGDATADIR=$(PTXCONF_SYSROOT_HOST)/share/bison
> +endif
> -- 
> 1.7.7.2
> 
> 
> -- 
> ptxdist mailing list
> ptxdist@pengutronix.de
> 

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

-- 
ptxdist mailing list
ptxdist@pengutronix.de