[Buildroot] [PATCH v3] pyrex: new package

Thomas Petazzoni thomas.petazzoni at free-electrons.com
Sat Mar 29 11:03:25 UTC 2014


Matt,

On Sat, 15 Mar 2014 17:00:30 -0500, mlweber1 at rockwellcollins.com wrote:

> diff --git a/package/python-pyrex/Config.in.host b/package/python-pyrex/Config.in.host
> new file mode 100644
> index 0000000..4edeb38
> --- /dev/null
> +++ b/package/python-pyrex/Config.in.host
> @@ -0,0 +1,10 @@
> +config BR2_PACKAGE_HOST_PYTHON_PYREX
> +	bool "python-pyrex"
> +	depends on BR2_PACKAGE_PYTHON

Is it actually really needed to have this host package visible in
menuconfig? python-pyrex is really a tool to generate code, so it
should be used by other packages as a dependency, and does not
necessarily need to be exposed in menuconfig.

To me, host-python-pyrex is a little bit like host-bison or host-flex:
useful as a dependency of another package, but not really useful as a
visible host package in menuconfig.

Would you agree if I removed this option when applying your patch?

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com


More information about the buildroot mailing list