[Buildroot] [PATCH] boost: Pass staging dir to --with-icu option.

[Buildroot] [PATCH] boost: Pass staging dir to --with-icu option.

[Ignacy Gawedzki]

When using the --with-icu option without specifying the directory, boost's
bootstrap.sh script will look at "common" locations (lines 289-294):

    COMMON_ICU_PATHS="/usr /usr/local /sw"
    for p in $COMMON_ICU_PATHS; do
      if test -r $p/include/unicode/utypes.h; then
        ICU_ROOT=$p
      fi
    done

With buildroot it may surely become problematic at some point.
---
 package/boost/boost.mk |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/boost/boost.mk b/package/boost/boost.mk
index 5e1bb4e..a202848 100644
--- a/package/boost/boost.mk
+++ b/package/boost/boost.mk
@@ -39,7 +39,7 @@ BOOST_WITHOUT_FLAGS += $(if $(BR2_PACKAGE_BOOST_TIMER),,timer)
 BOOST_WITHOUT_FLAGS += $(if $(BR2_PACKAGE_BOOST_WAVE),,wave)
 
 ifeq ($(BR2_PACKAGE_ICU),y)
-BOOST_FLAGS += --with-icu
+BOOST_FLAGS += --with-icu=$(STAGING_DIR)/usr
 BOOST_DEPENDENCIES += icu
 else
 BOOST_FLAGS += --without-icu
-- 
1.7.10.4

[Buildroot] [PATCH] boost: Pass staging dir to --with-icu option.

[Thomas Petazzoni]

Dear Ignacy Gawedzki,

On Thu, 7 Feb 2013 14:48:35 +0100, Ignacy Gawedzki wrote:
> When using the --with-icu option without specifying the directory,
> boost's bootstrap.sh script will look at "common" locations (lines
> 289-294):
> 
>     COMMON_ICU_PATHS="/usr /usr/local /sw"
>     for p in $COMMON_ICU_PATHS; do
>       if test -r $p/include/unicode/utypes.h; then
>         ICU_ROOT=$p
>       fi
>     done
> 
> With buildroot it may surely become problematic at some point.

This looks sensible. However, it doesn't explain how Boost could have
been finding ICU before your patch. Any idea?

Thomas
-- 
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com

[Buildroot] [PATCH] boost: Pass staging dir to --with-icu option.

[Ignacy Gawedzki]

On Thu, Feb 07, 2013 at 03:47:00PM +0100, thus spake Thomas Petazzoni:
> Dear Ignacy Gawedzki,
> 
> On Thu, 7 Feb 2013 14:48:35 +0100, Ignacy Gawedzki wrote:
> > When using the --with-icu option without specifying the directory,
> > boost's bootstrap.sh script will look at "common" locations (lines
> > 289-294):
> > 
> >     COMMON_ICU_PATHS="/usr /usr/local /sw"
> >     for p in $COMMON_ICU_PATHS; do
> >       if test -r $p/include/unicode/utypes.h; then
> >         ICU_ROOT=$p
> >       fi
> >     done
> > 
> > With buildroot it may surely become problematic at some point.
> 
> This looks sensible. However, it doesn't explain how Boost could have
> been finding ICU before your patch. Any idea?

The answer to this question may be in the following lines of bootstrap.sh.  If
ICU is not found in any of those "common" locations, then bjam is called
without any -sICU_PATH option and it may be that it finds the correct location
automatically (by using passed LDFLAGS or something).

I started using Boost with ICU very recently, so I can't tell from my own
experience if this is what happens indeed.

-- 
Information wants to be beer, or something like that.

[Buildroot] [PATCH] boost: Pass staging dir to --with-icu option.

[Thomas Petazzoni]

Dear Ignacy Gawedzki,

On Thu, 7 Feb 2013 16:24:26 +0100, Ignacy Gawedzki wrote:

> The answer to this question may be in the following lines of
> bootstrap.sh.  If ICU is not found in any of those "common"
> locations, then bjam is called without any -sICU_PATH option and it
> may be that it finds the correct location automatically (by using
> passed LDFLAGS or something).

Ok, makes sense. Thank you!

Best regards,

Thomas
-- 
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com

[Buildroot] [PATCH] boost: Pass staging dir to --with-icu option.

[Peter Korsgaard]

>>>>> "Ignacy" == Ignacy Gawedzki <i@lri.fr> writes:

 Ignacy> When using the --with-icu option without specifying the
 Ignacy> directory, boost's bootstrap.sh script will look at "common"
 Ignacy> locations (lines 289-294):

 Ignacy>     COMMON_ICU_PATHS="/usr /usr/local /sw"
 Ignacy>     for p in $COMMON_ICU_PATHS; do
 Ignacy>       if test -r $p/include/unicode/utypes.h; then
 Ignacy>         ICU_ROOT=$p
 Ignacy>       fi
 Ignacy>     done

 Ignacy> With buildroot it may surely become problematic at some point.
 Ignacy> ---

Can you please provide your signed-off-by so I can apply this? Thanks.

-- 
Bye, Peter Korsgaard

[Buildroot] [PATCH] boost: Pass staging dir to --with-icu option.

[Ignacy Gawedzki]

On Fri, Feb 08, 2013 at 10:27:35PM +0100, thus spake Peter Korsgaard:
> >>>>> "Ignacy" == Ignacy Gawedzki <i@lri.fr> writes:
> 
>  Ignacy> When using the --with-icu option without specifying the
>  Ignacy> directory, boost's bootstrap.sh script will look at "common"
>  Ignacy> loscript will look at "common"
>  Ignacy> locations (lines 289-294):
> 
>  Ignacy>     COMMON_ICU_PATHS="/usr /usr/local /sw"
>  Ignacy>     for p in $COMMON_ICU_PATHS; do
>  Ignacy>       if test -r $p/include/unicode/utypes.h; then
>  Ignacy>         ICU_ROOT=$p
>  Ignacy>       fi
>  Ignacy>     done
> 
>  Ignacy> With buildroot it may surely become problematic at some point.
>  Ignacy> ---
> 
> Can you please provide your signed-off-by so I can apply t

Do you mean this simple line:

  Signed-off-by: Ignacy Gaw?dzki <i@lri.fr>

?

-- 
A person is shit's way of making more shit.
		-- S. Barnett, anthropologist.

[Buildroot] [PATCH] boost: Pass staging dir to --with-icu option.

[Yann E. MORIN]

Ignacy, All,

On Friday 08 February 2013 Ignacy Gawedzki wrote:
> On Fri, Feb 08, 2013 at 10:27:35PM +0100, thus spake Peter Korsgaard:
> > Can you please provide your signed-off-by so I can apply t
> Do you mean this simple line:
>   Signed-off-by: Ignacy Gaw?dzki <i@lri.fr>

Yes, see: http://www.elinux.org/Developer_Certificate_Of_Origin

Regards,
Yann E. MORIN.

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'

[Buildroot] [PATCH] boost: Pass staging dir to --with-icu option.

[Peter Korsgaard]

>>>>> "Ignacy" == Ignacy Gawedzki <i@lri.fr> writes:

 Ignacy> On Fri, Feb 08, 2013 at 10:27:35PM +0100, thus spake Peter Korsgaard:
 >> >>>>> "Ignacy" == Ignacy Gawedzki <i@lri.fr> writes:
 >> 
 Ignacy> When using the --with-icu option without specifying the
 Ignacy> directory, boost's bootstrap.sh script will look at "common"
 Ignacy> loscript will look at "common"
 Ignacy> locations (lines 289-294):
 >> 
 Ignacy> COMMON_ICU_PATHS="/usr /usr/local /sw"
 Ignacy> for p in $COMMON_ICU_PATHS; do
 Ignacy> if test -r $p/include/unicode/utypes.h; then
 Ignacy> ICU_ROOT=$p
 Ignacy> fi
 Ignacy> done
 >> 
 Ignacy> With buildroot it may surely become problematic at some point.
 Ignacy> ---
 >> 
 >> Can you please provide your signed-off-by so I can apply t

 Ignacy> Do you mean this simple line:

 Ignacy>   Signed-off-by: Ignacy Gaw?dzki <i@lri.fr>

Yes, committed - Thanks.

-- 
Bye, Peter Korsgaard