Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 1/3] libevent: add host variant
@ 2013-10-14 14:15 Gustavo Zacarias
  2013-10-14 14:15 ` [Buildroot] [PATCH 2/3] boost: " Gustavo Zacarias
                   ` (2 more replies)
  0 siblings, 3 replies; 10+ messages in thread
From: Gustavo Zacarias @ 2013-10-14 14:15 UTC (permalink / raw)
  To: buildroot

Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
---
 package/libevent/libevent.mk | 1 +
 1 file changed, 1 insertion(+)

diff --git a/package/libevent/libevent.mk b/package/libevent/libevent.mk
index a6def1a..14ebf72 100644
--- a/package/libevent/libevent.mk
+++ b/package/libevent/libevent.mk
@@ -20,3 +20,4 @@ LIBEVENT_POST_INSTALL_TARGET_HOOKS += LIBEVENT_REMOVE_PYSCRIPT
 endif
 
 $(eval $(autotools-package))
+$(eval $(host-autotools-package))
-- 
1.8.1.5

^ permalink raw reply related	[flat|nested] 10+ messages in thread

* [Buildroot] [PATCH 2/3] boost: add host variant
  2013-10-14 14:15 [Buildroot] [PATCH 1/3] libevent: add host variant Gustavo Zacarias
@ 2013-10-14 14:15 ` Gustavo Zacarias
  2013-10-14 16:39   ` Thomas Petazzoni
  2013-10-14 14:15 ` [Buildroot] [PATCH 3/3] boost: fix build failures after version bump Gustavo Zacarias
  2013-10-14 16:39 ` [Buildroot] [PATCH 1/3] libevent: add host variant Thomas Petazzoni
  2 siblings, 1 reply; 10+ messages in thread
From: Gustavo Zacarias @ 2013-10-14 14:15 UTC (permalink / raw)
  To: buildroot

Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
---
 package/boost/boost.mk | 33 +++++++++++++++++++++++++++++++++
 1 file changed, 33 insertions(+)

diff --git a/package/boost/boost.mk b/package/boost/boost.mk
index a745ce1..8f2c2db 100644
--- a/package/boost/boost.mk
+++ b/package/boost/boost.mk
@@ -11,11 +11,19 @@ BOOST_SITE = http://downloads.sourceforge.net/project/boost/boost/$(BOOST_VERSIO
 BOOST_INSTALL_STAGING = YES
 
 TARGET_CC_VERSION = $(shell $(TARGET_CC) -dumpversion)
+HOST_CC_VERSION = $(shell $(HOSTCC) -dumpversion)
 
 BOOST_DEPENDENCIES =
 
 BOOST_FLAGS =
 
+# keep host variant as minimal as possible
+HOST_BOOST_FLAGS = --without-icu \
+	--without-libraries=$(subst $(space),$(comma),atomic chrono context \
+	coroutine date_time exception filesystem graph graph_parallel \
+	iostreams locale log math mpi program_options python random regex \
+	serialization signals system test thread timer wave)
+
 # atomic library compile only with upstream version, wait for next release
 BOOST_WITHOUT_FLAGS = python atomic
 
@@ -52,6 +60,9 @@ ifeq ($(BR2_PACKAGE_BOOST_IOSTREAMS),y)
 BOOST_DEPENDENCIES += bzip2 zlib
 endif
 
+HOST_BOOST_OPT += toolset=gcc threading=multi variant=release link=shared \
+	runtime-link=shared
+
 BOOST_OPT += toolset=gcc \
 	     threading=multi \
 	     variant=$(if $(BR2_ENABLE_DEBUG),debug,release) \
@@ -76,6 +87,12 @@ define BOOST_CONFIGURE_CMDS
 	echo "" >> $(@D)/user-config.jam
 endef
 
+define HOST_BOOST_CONFIGURE_CMDS
+	(cd $(@D) && ./bootstrap.sh $(HOST_BOOST_FLAGS))
+	echo "using gcc : $(HOST_CC_VERSION) : $(HOSTCXX) : <cxxflags>\"$(HOST_CXXFLAGS)\" <linkflags>\"$(HOST_LDFLAGS)\" ;" > $(@D)/user-config.jam
+	echo "" >> $(@D)/user-config.jam
+endef
+
 define BOOST_INSTALL_TARGET_CMDS
 	(cd $(@D) && ./b2 -j$(PARALLEL_JOBS) -q -d+1 \
 	--user-config=$(@D)/user-config.jam \
@@ -84,6 +101,21 @@ define BOOST_INSTALL_TARGET_CMDS
 	--layout=system install )
 endef
 
+define HOST_BOOST_BUILD_CMDS
+	(cd $(@D) && ./b2 -j$(PARALLEL_JOBS) -q -d+1 \
+	--user-config=$(@D)/user-config.jam \
+	$(HOST_BOOST_OPT) \
+	--prefix=$(HOST_DIR)/usr )
+endef
+
+define HOST_BOOST_INSTALL_CMDS
+	(cd $(@D) && ./b2 -j$(PARALLEL_JOBS) -q -d+1 \
+	--user-config=$(@D)/user-config.jam \
+	$(HOST_BOOST_OPT) \
+	--prefix=$(HOST_DIR)/usr \
+	--layout=system install )
+endef
+
 define BOOST_INSTALL_STAGING_CMDS
 	(cd $(@D) && ./bjam -j$(PARALLEL_JOBS) -d+1 \
 	--user-config=$(@D)/user-config.jam \
@@ -93,3 +125,4 @@ define BOOST_INSTALL_STAGING_CMDS
 endef
 
 $(eval $(generic-package))
+$(eval $(host-autotools-package))
-- 
1.8.1.5

^ permalink raw reply related	[flat|nested] 10+ messages in thread

* [Buildroot] [PATCH 3/3] boost: fix build failures after version bump
  2013-10-14 14:15 [Buildroot] [PATCH 1/3] libevent: add host variant Gustavo Zacarias
  2013-10-14 14:15 ` [Buildroot] [PATCH 2/3] boost: " Gustavo Zacarias
@ 2013-10-14 14:15 ` Gustavo Zacarias
  2013-10-14 20:26   ` Luca Ceresoli
  2013-10-14 20:49   ` Peter Korsgaard
  2013-10-14 16:39 ` [Buildroot] [PATCH 1/3] libevent: add host variant Thomas Petazzoni
  2 siblings, 2 replies; 10+ messages in thread
From: Gustavo Zacarias @ 2013-10-14 14:15 UTC (permalink / raw)
  To: buildroot

Fix build failures that happened after the version bump such as:
http://autobuild.buildroot.net/results/570/570b091702763b29843d9207bc14dea67085fea0/
http://autobuild.buildroot.net/results/c26/c26498f1a4e6bcbc3a2dfce6a51fa7d21b72f21f/
and other failures by disabling the new (1.54+) coroutine and log
libraries which weren't handled and hence enabled by default.

These also made the target size bigger and build times longer
unnecessarily.

When/if they are needed for some future user this can be revisited and
their proper conditions for enablement assesed.

Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
---
 package/boost/boost.mk | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/package/boost/boost.mk b/package/boost/boost.mk
index 8f2c2db..8a1a8f7 100644
--- a/package/boost/boost.mk
+++ b/package/boost/boost.mk
@@ -25,7 +25,9 @@ HOST_BOOST_FLAGS = --without-icu \
 	serialization signals system test thread timer wave)
 
 # atomic library compile only with upstream version, wait for next release
-BOOST_WITHOUT_FLAGS = python atomic
+# 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 python
 
 BOOST_WITHOUT_FLAGS += $(if $(BR2_PACKAGE_BOOST_CHRONO),,chrono)
 BOOST_WITHOUT_FLAGS += $(if $(BR2_PACKAGE_BOOST_CONTEXT),,context)
-- 
1.8.1.5

^ permalink raw reply related	[flat|nested] 10+ messages in thread

* [Buildroot] [PATCH 1/3] libevent: add host variant
  2013-10-14 14:15 [Buildroot] [PATCH 1/3] libevent: add host variant Gustavo Zacarias
  2013-10-14 14:15 ` [Buildroot] [PATCH 2/3] boost: " Gustavo Zacarias
  2013-10-14 14:15 ` [Buildroot] [PATCH 3/3] boost: fix build failures after version bump Gustavo Zacarias
@ 2013-10-14 16:39 ` Thomas Petazzoni
  2013-10-14 16:40   ` Gustavo Zacarias
  2 siblings, 1 reply; 10+ messages in thread
From: Thomas Petazzoni @ 2013-10-14 16:39 UTC (permalink / raw)
  To: buildroot

Dear Gustavo Zacarias,

On Mon, 14 Oct 2013 11:15:09 -0300, Gustavo Zacarias wrote:
> Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
> ---
>  package/libevent/libevent.mk | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/package/libevent/libevent.mk b/package/libevent/libevent.mk
> index a6def1a..14ebf72 100644
> --- a/package/libevent/libevent.mk
> +++ b/package/libevent/libevent.mk
> @@ -20,3 +20,4 @@ LIBEVENT_POST_INSTALL_TARGET_HOOKS += LIBEVENT_REMOVE_PYSCRIPT
>  endif
>  
>  $(eval $(autotools-package))
> +$(eval $(host-autotools-package))

Why?

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

^ permalink raw reply	[flat|nested] 10+ messages in thread

* [Buildroot] [PATCH 2/3] boost: add host variant
  2013-10-14 14:15 ` [Buildroot] [PATCH 2/3] boost: " Gustavo Zacarias
@ 2013-10-14 16:39   ` Thomas Petazzoni
  0 siblings, 0 replies; 10+ messages in thread
From: Thomas Petazzoni @ 2013-10-14 16:39 UTC (permalink / raw)
  To: buildroot

Dear Gustavo Zacarias,

On Mon, 14 Oct 2013 11:15:10 -0300, Gustavo Zacarias wrote:
> Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
> ---
>  package/boost/boost.mk | 33 +++++++++++++++++++++++++++++++++
>  1 file changed, 33 insertions(+)

Why?

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

^ permalink raw reply	[flat|nested] 10+ messages in thread

* [Buildroot] [PATCH 1/3] libevent: add host variant
  2013-10-14 16:39 ` [Buildroot] [PATCH 1/3] libevent: add host variant Thomas Petazzoni
@ 2013-10-14 16:40   ` Gustavo Zacarias
  2013-10-14 16:45     ` Thomas Petazzoni
  0 siblings, 1 reply; 10+ messages in thread
From: Gustavo Zacarias @ 2013-10-14 16:40 UTC (permalink / raw)
  To: buildroot

On 10/14/2013 01:39 PM, Thomas Petazzoni wrote:
> Why?

Apache thrift, upcoming package, same as host-boost.
Regards.

^ permalink raw reply	[flat|nested] 10+ messages in thread

* [Buildroot] [PATCH 1/3] libevent: add host variant
  2013-10-14 16:40   ` Gustavo Zacarias
@ 2013-10-14 16:45     ` Thomas Petazzoni
  2013-10-14 16:52       ` Gustavo Zacarias
  0 siblings, 1 reply; 10+ messages in thread
From: Thomas Petazzoni @ 2013-10-14 16:45 UTC (permalink / raw)
  To: buildroot

Dear Gustavo Zacarias,

On Mon, 14 Oct 2013 13:40:28 -0300, Gustavo Zacarias wrote:
> On 10/14/2013 01:39 PM, Thomas Petazzoni wrote:
> > Why?
> 
> Apache thrift, upcoming package, same as host-boost.

Then please say so, or include such packages in the patch series.
Otherwise, there's no way for us, mere reviewers, to know what's coming
next in your patch queue.

Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

^ permalink raw reply	[flat|nested] 10+ messages in thread

* [Buildroot] [PATCH 1/3] libevent: add host variant
  2013-10-14 16:45     ` Thomas Petazzoni
@ 2013-10-14 16:52       ` Gustavo Zacarias
  0 siblings, 0 replies; 10+ messages in thread
From: Gustavo Zacarias @ 2013-10-14 16:52 UTC (permalink / raw)
  To: buildroot

On 10/14/2013 01:45 PM, Thomas Petazzoni wrote:

>> Apache thrift, upcoming package, same as host-boost.
> 
> Then please say so, or include such packages in the patch series.
> Otherwise, there's no way for us, mere reviewers, to know what's coming
> next in your patch queue.

That's because i wanted to send the boost bump breakage fix and it was
on top of what i've already had done to boost.
Thrift needs some more testing and i didn't want to hold the fix back
(and also a filename collision fix with python-thrift because someone
thought it was wise to use the same naming convention for tarballs as
the upstream project for a subset of said thing).
Regards.

^ permalink raw reply	[flat|nested] 10+ messages in thread

* [Buildroot] [PATCH 3/3] boost: fix build failures after version bump
  2013-10-14 14:15 ` [Buildroot] [PATCH 3/3] boost: fix build failures after version bump Gustavo Zacarias
@ 2013-10-14 20:26   ` Luca Ceresoli
  2013-10-14 20:49   ` Peter Korsgaard
  1 sibling, 0 replies; 10+ messages in thread
From: Luca Ceresoli @ 2013-10-14 20:26 UTC (permalink / raw)
  To: buildroot

Hi Gustavo,

Gustavo Zacarias wrote:
> Fix build failures that happened after the version bump such as:
> http://autobuild.buildroot.net/results/570/570b091702763b29843d9207bc14dea67085fea0/
> http://autobuild.buildroot.net/results/c26/c26498f1a4e6bcbc3a2dfce6a51fa7d21b72f21f/
> and other failures by disabling the new (1.54+) coroutine and log
> libraries which weren't handled and hence enabled by default.
>
> These also made the target size bigger and build times longer
> unnecessarily.
>
> When/if they are needed for some future user this can be revisited and
> their proper conditions for enablement assesed.
>
> Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>

This one bit me as well, thanks for the fix.

Acked-by: Luca Ceresoli <luca@lucaceresoli.net>

-- 
Luca

^ permalink raw reply	[flat|nested] 10+ messages in thread

* [Buildroot] [PATCH 3/3] boost: fix build failures after version bump
  2013-10-14 14:15 ` [Buildroot] [PATCH 3/3] boost: fix build failures after version bump Gustavo Zacarias
  2013-10-14 20:26   ` Luca Ceresoli
@ 2013-10-14 20:49   ` Peter Korsgaard
  1 sibling, 0 replies; 10+ messages in thread
From: Peter Korsgaard @ 2013-10-14 20:49 UTC (permalink / raw)
  To: buildroot

>>>>> "Gustavo" == Gustavo Zacarias <gustavo@zacarias.com.ar> writes:

 Gustavo> Fix build failures that happened after the version bump such as:
 Gustavo> http://autobuild.buildroot.net/results/570/570b091702763b29843d9207bc14dea67085fea0/
 Gustavo> http://autobuild.buildroot.net/results/c26/c26498f1a4e6bcbc3a2dfce6a51fa7d21b72f21f/
 Gustavo> and other failures by disabling the new (1.54+) coroutine and log
 Gustavo> libraries which weren't handled and hence enabled by default.

 Gustavo> These also made the target size bigger and build times longer
 Gustavo> unnecessarily.

 Gustavo> When/if they are needed for some future user this can be revisited and
 Gustavo> their proper conditions for enablement assesed.

Committed, thanks.

-- 
Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2013-10-14 20:49 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-10-14 14:15 [Buildroot] [PATCH 1/3] libevent: add host variant Gustavo Zacarias
2013-10-14 14:15 ` [Buildroot] [PATCH 2/3] boost: " Gustavo Zacarias
2013-10-14 16:39   ` Thomas Petazzoni
2013-10-14 14:15 ` [Buildroot] [PATCH 3/3] boost: fix build failures after version bump Gustavo Zacarias
2013-10-14 20:26   ` Luca Ceresoli
2013-10-14 20:49   ` Peter Korsgaard
2013-10-14 16:39 ` [Buildroot] [PATCH 1/3] libevent: add host variant Thomas Petazzoni
2013-10-14 16:40   ` Gustavo Zacarias
2013-10-14 16:45     ` Thomas Petazzoni
2013-10-14 16:52       ` Gustavo Zacarias

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox