All of lore.kernel.org
 help / color / mirror / Atom feed
From: Will Wagner <will_wagner@carallon.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH v2] boost: only the iostreams library requires bzip2
Date: Tue, 11 Jun 2013 07:55:07 +0100	[thread overview]
Message-ID: <51B6C9CB.3010802@carallon.com> (raw)
In-Reply-To: <51B6C902.6000101@carallon.com>

On 11/06/2013 07:51, Will Wagner wrote:
> Signed-off-by: Will Wagner <will_wagner@carallon.com>
> ---
>   package/boost/Config.in |    2 +-
>   package/boost/boost.mk  |    6 +++++-
>   2 files changed, 6 insertions(+), 2 deletions(-)
>
> diff --git a/package/boost/Config.in b/package/boost/Config.in
> index 5c65b2f..46b5102 100644
> --- a/package/boost/Config.in
> +++ b/package/boost/Config.in
> @@ -9,7 +9,6 @@ config BR2_PACKAGE_BOOST
>       # that unfortunately doesn't work. Until someone fixes that,
>       # let's depend on threads.
>       depends on BR2_TOOLCHAIN_HAS_THREADS
> -    select BR2_PACKAGE_BZIP2
>       select BR2_PACKAGE_ZLIB
>       help
>         A general purpose C++ library
> @@ -43,6 +42,7 @@ config BR2_PACKAGE_BOOST_GRAPH_PARALLEL
>
>   config BR2_PACKAGE_BOOST_IOSTREAMS
>       bool "boost-iostreams"
> +    select BR2_PACKAGE_BZIP2
>
>   config BR2_PACKAGE_BOOST_LOCALE
>       depends on BR2_USE_WCHAR
> diff --git a/package/boost/boost.mk b/package/boost/boost.mk
> index d11f378..73eb4b0 100644
> --- a/package/boost/boost.mk
> +++ b/package/boost/boost.mk
> @@ -12,7 +12,7 @@ BOOST_INSTALL_STAGING = YES
>
>   TARGET_CC_VERSION = $(shell $(TARGET_CC) -dumpversion)
>
> -BOOST_DEPENDENCIES = bzip2 zlib
> +BOOST_DEPENDENCIES = zlib
>
>   BOOST_FLAGS =
>
> @@ -48,6 +48,10 @@ else
>   BOOST_FLAGS += --without-icu
>   endif
>
> +ifeq ($(BR2_PACKAGE_BOOST_IOSTREAMS),y)
> +BOOST_DEPENDENCIES += bzip2
> +endif
> +
>   BOOST_OPT += toolset=gcc \
>            threading=multi \
>            variant=$(if $(BR2_ENABLE_DEBUG),debug,release) \


Having just looked at the boost source code again I realise that zlib is 
also only needed for iostreams lib. I'll update the patch and resubmit.

Regards
Will

-- 
------------------------------------------------------------------------
Will Wagner                                     will_wagner at carallon.com
Development Manager                      Office Tel: +44 (0)20 7471 9224
Carallon Ltd, Studio G20, Shepherds Building, Rockley Rd, London W14 0DA
------------------------------------------------------------------------

      reply	other threads:[~2013-06-11  6:55 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-06-11  6:51 [Buildroot] [PATCH v2] boost: only the iostreams library requires bzip2 Will Wagner
2013-06-11  6:55 ` Will Wagner [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=51B6C9CB.3010802@carallon.com \
    --to=will_wagner@carallon.com \
    --cc=buildroot@busybox.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.