[Buildroot] [PATCH] log4cxx: needs threads

Thomas De Schampheleire patrickdepinguin at gmail.com
Mon Nov 11 13:30:42 UTC 2013


Hi Simon,

On Mon, Nov 11, 2013 at 2:14 PM,  <spdawson at gmail.com> wrote:
> From: Simon Dawson <spdawson at gmail.com>
>
> Fixes build failures such as the following.
>
>   http://autobuild.buildroot.net/results/7b73901d9e8b5098cd96b54d292bb3ffffb3fe37/
>
> Signed-off-by: Simon Dawson <spdawson at gmail.com>
> ---
>  package/log4cxx/Config.in | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/package/log4cxx/Config.in b/package/log4cxx/Config.in
> index 86d6a1d..1785232 100644
> --- a/package/log4cxx/Config.in
> +++ b/package/log4cxx/Config.in
> @@ -5,11 +5,12 @@ config BR2_PACKAGE_LOG4CXX
>         # apr really needs shared library support
>         depends on !BR2_PREFER_STATIC_LIB
>         depends on BR2_INSTALL_LIBSTDCPP
> +       depends on BR2_TOOLCHAIN_HAS_THREADS
>         help
>           Apache log4cxx is a logging framework for C++ patterned
>           after Apache log4j.
>
>           http://apache.linux-mirror.org/logging/log4cxx/0.10.0
>
> -comment "log4cxx needs a toolchain w/ C++"
> -       depends on !BR2_INSTALL_LIBSTDCPP
> +comment "log4cxx needs a toolchain w/ C++, threads"
> +       depends on !(BR2_INSTALL_LIBSTDCPP && BR2_TOOLCHAIN_HAS_THREADS)
> --

Maybe you could take the opportunity to update the comment for the
dynamic library support needed. See the buildroot manual (git version)
for details.

Thanks,
Thomas


More information about the buildroot mailing list