[Buildroot] [git commit] beecrypt: only build C++ support if dependencies are available
Arnout Vandecappelle
arnout at mind.be
Wed Mar 26 17:36:05 UTC 2014
On 26/03/14 16:01, Peter Korsgaard wrote:
> commit: http://git.buildroot.net/buildroot/commit/?id=205839d085508b83d9437f2fbe002ef076aa0cdb
> branch: http://git.buildroot.net/buildroot/commit/?id=refs/heads/master
>
> Instead of automatically pulling in the (big) icu library if the toolchain
> dependencies are available.
Wouldn't it be more appropriate to make an explicit config option for it
then? I think this one falls under the category "Not obvious to the user
that he should select library XXX to enable feature YYY."
Patch follows.
Regards,
Arnout
>
> Signed-off-by: Peter Korsgaard <peter at korsgaard.com>
> ---
> package/beecrypt/Config.in | 7 -------
> package/beecrypt/beecrypt.mk | 2 +-
> 2 files changed, 1 insertions(+), 8 deletions(-)
>
> diff --git a/package/beecrypt/Config.in b/package/beecrypt/Config.in
> index 8d63e5e..84754e5 100644
> --- a/package/beecrypt/Config.in
> +++ b/package/beecrypt/Config.in
> @@ -1,8 +1,6 @@
> config BR2_PACKAGE_BEECRYPT
> bool "beecrypt"
> depends on BR2_TOOLCHAIN_HAS_THREADS
> - select BR2_PACKAGE_ICU if BR2_INSTALL_LIBSTDCPP && BR2_USE_WCHAR && \
> - !BR2_arc && !BR2_BINFMT_FLAT
> help
> Beecrypt is a general-purpose cryptography library.
>
> @@ -10,8 +8,3 @@ config BR2_PACKAGE_BEECRYPT
>
> comment "beecrypt needs a toolchain w/ threads"
> depends on !BR2_TOOLCHAIN_HAS_THREADS
> -
> -comment "beecrypt C++ support needs a toolchain w/ wchar"
> - depends on !BR2_arc
> - depends on !BR2_BINFMT_FLAT
> - depends on BR2_INSTALL_LIBSTDCPP && !BR2_USE_WCHAR && BR2_TOOLCHAIN_HAS_THREADS
> diff --git a/package/beecrypt/beecrypt.mk b/package/beecrypt/beecrypt.mk
> index 223b44f..c77782c 100644
> --- a/package/beecrypt/beecrypt.mk
> +++ b/package/beecrypt/beecrypt.mk
> @@ -22,7 +22,7 @@ BEECRYPT_CONF_OPT = \
> --without-python \
> --disable-openmp
>
> -ifeq ($(BR2_PACKAGE_ICU),y)
> +ifeq ($(BR2_INSTALL_LIBSTDCPP)$(BR2_USE_WCHAR)$(BR2_PACKAGE_ICU),yyy)
> # C++ support needs icu
> BEECRYPT_DEPENDENCIES += icu
> else
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
>
--
Arnout Vandecappelle arnout at mind be
Senior Embedded Software Architect +32-16-286500
Essensium/Mind http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F
More information about the buildroot
mailing list