[Buildroot] [PATCH 1/3] package/beecrypt: add host variant
Thomas Petazzoni
thomas.petazzoni at free-electrons.com
Tue Aug 18 09:43:24 UTC 2015
James,
On Thu, 13 Aug 2015 13:14:49 -0400, James Knight wrote:
> Required by an upcoming host variant of the `rpm` package.
>
> Signed-off-by: James Knight <james.knight at rockwellcollins.com>
> ---
> package/beecrypt/beecrypt.mk | 9 ++++++++-
> 1 file changed, 8 insertions(+), 1 deletion(-)
>
> diff --git a/package/beecrypt/beecrypt.mk b/package/beecrypt/beecrypt.mk
> index cd0b4ac..0d7b360 100644
> --- a/package/beecrypt/beecrypt.mk
> +++ b/package/beecrypt/beecrypt.mk
> @@ -11,16 +11,21 @@ BEECRYPT_INSTALL_STAGING = YES
> BEECRYPT_LICENSE = LGPLv2.1+
> BEECRYPT_LICENSE_FILES = COPYING.LIB
>
> -BEECRYPT_CONF_OPTS = \
> +BEECRYPT_CONFIGURE_ARGS = \
> --disable-expert-mode \
> --without-java \
> --without-python \
> --disable-openmp
>
> +BEECRYPT_CONF_OPTS = $(BEECRYPT_CONFIGURE_ARGS)
> +HOST_BEECRYPT_CONF_OPTS = $(BEECRYPT_CONFIGURE_ARGS)
> +
> ifeq ($(BR2_PACKAGE_BEECRYPT_CPP),y)
> BEECRYPT_DEPENDENCIES += icu
> +HOST_BEECRYPT_DEPENDENCIES += host-icu
This is really wrong. BR2_PACKAGE_BEECRYPT_CPP indicates that we want
the C++ support of Beecrypt for the target. It says nothing about what
should be done for the host variant of Beecrypt.
Target configuration options (BR2_PACKAGE_*) should not affect how host
packages are built.
Just decide if you need C++ support in host-beecrypt. If you need it,
enable it. If you don't, do not enable it. But don't make it
conditional.
Thanks,
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
More information about the buildroot
mailing list