[Buildroot] [PATCH] bonnie: does not support static builds
Thomas Petazzoni
thomas.petazzoni at free-electrons.com
Tue Dec 17 11:41:49 UTC 2013
Simon,
On Tue, 17 Dec 2013 11:28:38 +0000, spdawson at gmail.com wrote:
> From: Simon Dawson <spdawson at gmail.com>
>
> Fixes autobuild failues such as the following.
>
> http://autobuild.buildroot.net/results/bb9/bb9665a957e4fed25dff3e36bbf105c523dcf0a7/
>
> Signed-off-by: Simon Dawson <spdawson at gmail.com>
> ---
> package/bonnie/Config.in | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/package/bonnie/Config.in b/package/bonnie/Config.in
> index 6dbd06d..a4a50f8 100644
> --- a/package/bonnie/Config.in
> +++ b/package/bonnie/Config.in
> @@ -2,11 +2,12 @@ config BR2_PACKAGE_BONNIE
> bool "bonnie++"
> depends on BR2_INSTALL_LIBSTDCPP
> depends on BR2_USE_MMU # fork()
> + depends on !BR2_PREFER_STATIC_LIB
> help
> Filesystem tester
>
> http://www.coker.com.au/bonnie++/
>
> -comment "bonnie++ needs a toolchain w/ C++"
> +comment "bonnie++ needs a toolchain w/ C++, dynamic library"
> depends on BR2_USE_MMU
> - depends on !BR2_INSTALL_LIBSTDCPP
> + depends on !BR2_INSTALL_LIBSTDCPP || BR2_PREFER_STATIC_LIB
I need to test this specific case, but I don't think this is the
correct fix. If you look at the config.log file, I'm pretty sure you
will see that configure bails out because libstdc++.a does not exist.
This is a problem in the internal toolchain rework I've done, and that
I need to have a look at (make sure it generates the static version of
libstdc++).
Therefore, I don't think this patch should be applied: bonnie++, by
itself, does not require dynamic library.
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