* [Buildroot] [PATCH 1/1] package/boost: enable building of boost_log
@ 2014-10-20 13:27 Yannick Kiekens
2014-10-20 17:02 ` Yann E. MORIN
2015-03-15 16:49 ` Thomas Petazzoni
0 siblings, 2 replies; 6+ messages in thread
From: Yannick Kiekens @ 2014-10-20 13:27 UTC (permalink / raw)
To: buildroot
Signed-off-by: Yannick Kiekens <yannickkiekens@gmail.com>
---
package/boost/Config.in | 3 +++
package/boost/boost.mk | 4 ++--
2 files changed, 5 insertions(+), 2 deletions(-)
diff --git a/package/boost/Config.in b/package/boost/Config.in
index a15b849..5ac1d0e 100644
--- a/package/boost/Config.in
+++ b/package/boost/Config.in
@@ -85,6 +85,9 @@ config BR2_PACKAGE_BOOST_LOCALE
comment "boost-locale needs a toolchain w/ wchar"
depends on !BR2_USE_WCHAR
+config BR2_PACKAGE_BOOST_LOG
+ bool "boost-log"
+
config BR2_PACKAGE_BOOST_MATH
bool "boost-math"
diff --git a/package/boost/boost.mk b/package/boost/boost.mk
index 46a98e8..06969b4 100644
--- a/package/boost/boost.mk
+++ b/package/boost/boost.mk
@@ -25,8 +25,7 @@ HOST_BOOST_FLAGS = --without-icu \
# atomic library compile only with upstream version, wait for next release
# coroutine breaks on some weak toolchains and it's new for 1.54+
-# log breaks with some toolchain combinations and it's new for 1.54+
-BOOST_WITHOUT_FLAGS = atomic coroutine log
+BOOST_WITHOUT_FLAGS = atomic coroutine
BOOST_WITHOUT_FLAGS += $(if $(BR2_PACKAGE_BOOST_CHRONO),,chrono)
BOOST_WITHOUT_FLAGS += $(if $(BR2_PACKAGE_BOOST_CONTEXT),,context)
@@ -37,6 +36,7 @@ BOOST_WITHOUT_FLAGS += $(if $(BR2_PACKAGE_BOOST_GRAPH),,graph)
BOOST_WITHOUT_FLAGS += $(if $(BR2_PACKAGE_BOOST_GRAPH_PARALLEL),,graph_parallel)
BOOST_WITHOUT_FLAGS += $(if $(BR2_PACKAGE_BOOST_IOSTREAMS),,iostreams)
BOOST_WITHOUT_FLAGS += $(if $(BR2_PACKAGE_BOOST_LOCALE),,locale)
+BOOST_WITHOUT_FLAGS += $(if $(BR2_PACKAGE_BOOST_LOG),,log)
BOOST_WITHOUT_FLAGS += $(if $(BR2_PACKAGE_BOOST_MATH),,math)
BOOST_WITHOUT_FLAGS += $(if $(BR2_PACKAGE_BOOST_MPI),,mpi)
BOOST_WITHOUT_FLAGS += $(if $(BR2_PACKAGE_BOOST_PROGRAM_OPTIONS),,program_options)
--
1.9.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [Buildroot] [PATCH 1/1] package/boost: enable building of boost_log
2014-10-20 13:27 [Buildroot] [PATCH 1/1] package/boost: enable building of boost_log Yannick Kiekens
@ 2014-10-20 17:02 ` Yann E. MORIN
2014-10-20 17:07 ` Yann E. MORIN
2015-03-15 16:49 ` Thomas Petazzoni
1 sibling, 1 reply; 6+ messages in thread
From: Yann E. MORIN @ 2014-10-20 17:02 UTC (permalink / raw)
To: buildroot
Yannick, All,
On 2014-10-20 15:27 +0200, Yannick Kiekens spake thusly:
> Signed-off-by: Yannick Kiekens <yannickkiekens@gmail.com>
> ---
> package/boost/Config.in | 3 +++
> package/boost/boost.mk | 4 ++--
> 2 files changed, 5 insertions(+), 2 deletions(-)
>
> diff --git a/package/boost/Config.in b/package/boost/Config.in
> index a15b849..5ac1d0e 100644
> --- a/package/boost/Config.in
> +++ b/package/boost/Config.in
> @@ -85,6 +85,9 @@ config BR2_PACKAGE_BOOST_LOCALE
> comment "boost-locale needs a toolchain w/ wchar"
> depends on !BR2_USE_WCHAR
>
> +config BR2_PACKAGE_BOOST_LOG
> + bool "boost-log"
> +
> config BR2_PACKAGE_BOOST_MATH
> bool "boost-math"
>
> diff --git a/package/boost/boost.mk b/package/boost/boost.mk
> index 46a98e8..06969b4 100644
> --- a/package/boost/boost.mk
> +++ b/package/boost/boost.mk
> @@ -25,8 +25,7 @@ HOST_BOOST_FLAGS = --without-icu \
>
> # atomic library compile only with upstream version, wait for next release
> # coroutine breaks on some weak toolchains and it's new for 1.54+
> -# log breaks with some toolchain combinations and it's new for 1.54+
As you remove a comment that warns that log is broken in some cases, it
would be nice if you were to explain why this comment no longer applies.
For example, you could format your commit log as thus (reason totally
made up for the sake of the example, adapt):
package/boost: enable building of boost_log
In the current version of boost, boost_log is now considered stable
and maintained. The failures caused by "some toolchain combinations"
no longer occur.
Signed-off-by: [YOU]
---
Tested with these toolchains:
- uClibc, arm, static only
- uClibc, x86_64
Note the '---' line: you can add such a line in your commit log, and it
will appear as-is in the mail you send. However, git is smart enough so
that when the maintainer applies the patch, git removes it.
Otherwise, I'm no boost expert, but the patch looks OK.
Regards,
Yann E. MORIN.
> -BOOST_WITHOUT_FLAGS = atomic coroutine log
> +BOOST_WITHOUT_FLAGS = atomic coroutine
>
> BOOST_WITHOUT_FLAGS += $(if $(BR2_PACKAGE_BOOST_CHRONO),,chrono)
> BOOST_WITHOUT_FLAGS += $(if $(BR2_PACKAGE_BOOST_CONTEXT),,context)
> @@ -37,6 +36,7 @@ BOOST_WITHOUT_FLAGS += $(if $(BR2_PACKAGE_BOOST_GRAPH),,graph)
> BOOST_WITHOUT_FLAGS += $(if $(BR2_PACKAGE_BOOST_GRAPH_PARALLEL),,graph_parallel)
> BOOST_WITHOUT_FLAGS += $(if $(BR2_PACKAGE_BOOST_IOSTREAMS),,iostreams)
> BOOST_WITHOUT_FLAGS += $(if $(BR2_PACKAGE_BOOST_LOCALE),,locale)
> +BOOST_WITHOUT_FLAGS += $(if $(BR2_PACKAGE_BOOST_LOG),,log)
> BOOST_WITHOUT_FLAGS += $(if $(BR2_PACKAGE_BOOST_MATH),,math)
> BOOST_WITHOUT_FLAGS += $(if $(BR2_PACKAGE_BOOST_MPI),,mpi)
> BOOST_WITHOUT_FLAGS += $(if $(BR2_PACKAGE_BOOST_PROGRAM_OPTIONS),,program_options)
> --
> 1.9.1
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
--
.-----------------.--------------------.------------------.--------------------.
| 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. |
'------------------------------^-------^------------------^--------------------'
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Buildroot] [PATCH 1/1] package/boost: enable building of boost_log
2014-10-20 17:02 ` Yann E. MORIN
@ 2014-10-20 17:07 ` Yann E. MORIN
2014-10-21 7:15 ` Yannick Kiekens
0 siblings, 1 reply; 6+ messages in thread
From: Yann E. MORIN @ 2014-10-20 17:07 UTC (permalink / raw)
To: buildroot
Yannick, All,
On 2014-10-20 19:02 +0200, Yann E. MORIN spake thusly:
> On 2014-10-20 15:27 +0200, Yannick Kiekens spake thusly:
[--SNIP--]
> > diff --git a/package/boost/boost.mk b/package/boost/boost.mk
> > index 46a98e8..06969b4 100644
> > --- a/package/boost/boost.mk
> > +++ b/package/boost/boost.mk
> > @@ -25,8 +25,7 @@ HOST_BOOST_FLAGS = --without-icu \
> >
> > # atomic library compile only with upstream version, wait for next release
> > # coroutine breaks on some weak toolchains and it's new for 1.54+
> > -# log breaks with some toolchain combinations and it's new for 1.54+
>
> As you remove a comment that warns that log is broken in some cases, it
> would be nice if you were to explain why this comment no longer applies.
>
> For example, you could format your commit log as thus (reason totally
> made up for the sake of the example, adapt):
>
> package/boost: enable building of boost_log
>
> In the current version of boost, boost_log is now considered stable
> and maintained. The failures caused by "some toolchain combinations"
> no longer occur.
>
> Signed-off-by: [YOU]
>
> ---
> Tested with these toolchains:
> - uClibc, arm, static only
> - uClibc, x86_64
>
> Note the '---' line: you can add such a line in your commit log, and it
> will appear as-is in the mail you send. However, git is smart enough so
> that when the maintainer applies the patch, git removes it.
Sorry, I meant:
[...] git removes it, and whatever follows (but keeps the patch, of
course).
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. |
'------------------------------^-------^------------------^--------------------'
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Buildroot] [PATCH 1/1] package/boost: enable building of boost_log
2014-10-20 17:07 ` Yann E. MORIN
@ 2014-10-21 7:15 ` Yannick Kiekens
2014-10-21 9:56 ` Thomas Petazzoni
0 siblings, 1 reply; 6+ messages in thread
From: Yannick Kiekens @ 2014-10-21 7:15 UTC (permalink / raw)
To: buildroot
On Mon, Oct 20, 2014 at 7:07 PM, Yann E. MORIN <yann.morin.1998@free.fr>
wrote:
> Yannick, All,
>
> On 2014-10-20 19:02 +0200, Yann E. MORIN spake thusly:
> > On 2014-10-20 15:27 +0200, Yannick Kiekens spake thusly:
> [--SNIP--]
> > > diff --git a/package/boost/boost.mk b/package/boost/boost.mk
> > > index 46a98e8..06969b4 100644
> > > --- a/package/boost/boost.mk
> > > +++ b/package/boost/boost.mk
> > > @@ -25,8 +25,7 @@ HOST_BOOST_FLAGS = --without-icu \
> > >
> > > # atomic library compile only with upstream version, wait for next
> release
> > > # coroutine breaks on some weak toolchains and it's new for 1.54+
> > > -# log breaks with some toolchain combinations and it's new for 1.54+
> >
> > As you remove a comment that warns that log is broken in some cases, it
> > would be nice if you were to explain why this comment no longer applies.
> >
> > For example, you could format your commit log as thus (reason totally
> > made up for the sake of the example, adapt):
> >
> > package/boost: enable building of boost_log
> >
> > In the current version of boost, boost_log is now considered stable
> > and maintained. The failures caused by "some toolchain combinations"
> > no longer occur.
> >
> > Signed-off-by: [YOU]
> >
> > ---
> > Tested with these toolchains:
> > - uClibc, arm, static only
> > - uClibc, x86_64
> >
> > Note the '---' line: you can add such a line in your commit log, and it
> > will appear as-is in the mail you send. However, git is smart enough so
> > that when the maintainer applies the patch, git removes it.
>
> Sorry, I meant:
> [...] git removes it, and whatever follows (but keeps the patch, of
> course).
>
> Regards,
> Yann E. MORIN.
>
>
?We use boost_log at work on x86_64 and armv7, we never had trouble with it.
It is possible boost_log 1.56 still does not work on some toolchain
combinations?, as mentioned in git
commit 611dfe6cb55ed72d7df3b7c102f4a723248a4037
?I could reformat the patch to whitelist boost_log on known configurations
Any opnion on that?
Regards,
Yannick Kiekens?
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20141021/5b9320a5/attachment.html>
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Buildroot] [PATCH 1/1] package/boost: enable building of boost_log
2014-10-21 7:15 ` Yannick Kiekens
@ 2014-10-21 9:56 ` Thomas Petazzoni
0 siblings, 0 replies; 6+ messages in thread
From: Thomas Petazzoni @ 2014-10-21 9:56 UTC (permalink / raw)
To: buildroot
Dear Yannick Kiekens,
On Tue, 21 Oct 2014 09:15:24 +0200, Yannick Kiekens wrote:
> ?We use boost_log at work on x86_64 and armv7, we never had trouble with it.
> It is possible boost_log 1.56 still does not work on some toolchain
> combinations?, as mentioned in git
> commit 611dfe6cb55ed72d7df3b7c102f4a723248a4037
>
> ?I could reformat the patch to whitelist boost_log on known configurations
> Any opnion on that?
Can you try looking at
http://autobuild.buildroot.net/results/c26/c26498f1a4e6bcbc3a2dfce6a51fa7d21b72f21f/build-end.log,
which is the issue reported in commit
611dfe6cb55ed72d7df3b7c102f4a723248a4037 ?
Best regards,
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Buildroot] [PATCH 1/1] package/boost: enable building of boost_log
2014-10-20 13:27 [Buildroot] [PATCH 1/1] package/boost: enable building of boost_log Yannick Kiekens
2014-10-20 17:02 ` Yann E. MORIN
@ 2015-03-15 16:49 ` Thomas Petazzoni
1 sibling, 0 replies; 6+ messages in thread
From: Thomas Petazzoni @ 2015-03-15 16:49 UTC (permalink / raw)
To: buildroot
Dear Yannick Kiekens,
On Mon, 20 Oct 2014 15:27:24 +0200, Yannick Kiekens wrote:
> Signed-off-by: Yannick Kiekens <yannickkiekens@gmail.com>
> ---
> package/boost/Config.in | 3 +++
> package/boost/boost.mk | 4 ++--
> 2 files changed, 5 insertions(+), 2 deletions(-)
Applied, thanks!
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2015-03-15 16:49 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-10-20 13:27 [Buildroot] [PATCH 1/1] package/boost: enable building of boost_log Yannick Kiekens
2014-10-20 17:02 ` Yann E. MORIN
2014-10-20 17:07 ` Yann E. MORIN
2014-10-21 7:15 ` Yannick Kiekens
2014-10-21 9:56 ` Thomas Petazzoni
2015-03-15 16:49 ` Thomas Petazzoni
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox