Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 0/2] Fix build of shared libraries
@ 2011-06-20 15:59 Luca Ceresoli
  2011-06-20 15:59 ` [Buildroot] [PATCH 1/2] ffmpeg: fix " Luca Ceresoli
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Luca Ceresoli @ 2011-06-20 15:59 UTC (permalink / raw)
  To: buildroot

Hi,

009d8fceab4 and 0a5e5534cbf8 broke shared library building for those AUTOTOOLS
packaged that override _CONFIGURE_CMDS.

According to my grep'ing, the affetced packages should be only ffmpeg and
berkeleydb. These two patched fixes them.

Please note I only could test ffmpeg, but the change is pretty trivial so
hopefully it works for berkeleydb as well.

Luca

Luca Ceresoli (2):
  ffmpeg: fix build of shared libraries
  berkeleydb: fix build of shared libraries

 package/berkeleydb/berkeleydb.mk    |    1 +
 package/multimedia/ffmpeg/ffmpeg.mk |    1 +
 2 files changed, 2 insertions(+), 0 deletions(-)

-- 
1.7.4.1

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

* [Buildroot] [PATCH 1/2] ffmpeg: fix build of shared libraries
  2011-06-20 15:59 [Buildroot] [PATCH 0/2] Fix build of shared libraries Luca Ceresoli
@ 2011-06-20 15:59 ` Luca Ceresoli
  2011-06-20 15:59 ` [Buildroot] [PATCH 2/2] berkeleydb: " Luca Ceresoli
  2011-06-21 11:29 ` [Buildroot] [PATCH 0/2] Fix " Peter Korsgaard
  2 siblings, 0 replies; 4+ messages in thread
From: Luca Ceresoli @ 2011-06-20 15:59 UTC (permalink / raw)
  To: buildroot

009d8fceab4 and 0a5e5534cbf8 moved the --enable-static configure flag
from package files to the default _CONFIGURE_CMDS in
package/Makefile.autotools.in.
This broke Autotools packages that override _CONFIGURE_CMDS.

Signed-off-by: Luca Ceresoli <luca@lucaceresoli.net>
---
 package/multimedia/ffmpeg/ffmpeg.mk |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/package/multimedia/ffmpeg/ffmpeg.mk b/package/multimedia/ffmpeg/ffmpeg.mk
index babb16a..b50503b 100644
--- a/package/multimedia/ffmpeg/ffmpeg.mk
+++ b/package/multimedia/ffmpeg/ffmpeg.mk
@@ -167,6 +167,7 @@ define FFMPEG_CONFIGURE_CMDS
 		--arch=$(BR2_ARCH) \
 		--target-os=linux \
 		--extra-cflags=-fPIC \
+		$(SHARED_STATIC_LIBS_OPTS) \
 		$(FFMPEG_CONF_OPT) \
 	)
 endef
-- 
1.7.4.1

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

* [Buildroot] [PATCH 2/2] berkeleydb: fix build of shared libraries
  2011-06-20 15:59 [Buildroot] [PATCH 0/2] Fix build of shared libraries Luca Ceresoli
  2011-06-20 15:59 ` [Buildroot] [PATCH 1/2] ffmpeg: fix " Luca Ceresoli
@ 2011-06-20 15:59 ` Luca Ceresoli
  2011-06-21 11:29 ` [Buildroot] [PATCH 0/2] Fix " Peter Korsgaard
  2 siblings, 0 replies; 4+ messages in thread
From: Luca Ceresoli @ 2011-06-20 15:59 UTC (permalink / raw)
  To: buildroot

009d8fceab4 and 0a5e5534cbf8 moved the --enable-static configure flag
from package files to the default _CONFIGURE_CMDS in
package/Makefile.autotools.in.
This broke Autotools packages that override _CONFIGURE_CMDS.

Signed-off-by: Luca Ceresoli <luca@lucaceresoli.net>
---
 package/berkeleydb/berkeleydb.mk |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/package/berkeleydb/berkeleydb.mk b/package/berkeleydb/berkeleydb.mk
index 1bac745..e80f66f 100644
--- a/package/berkeleydb/berkeleydb.mk
+++ b/package/berkeleydb/berkeleydb.mk
@@ -38,6 +38,7 @@ define BERKELEYDB_CONFIGURE_CMDS
 		--disable-rpc \
 		--disable-tcl \
 		--disable-compat185 \
+		$(SHARED_STATIC_LIBS_OPTS) \
 		--with-pic \
 	)
 	$(SED) 's/\.lo/.o/g' $(@D)/build_unix/Makefile
-- 
1.7.4.1

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

* [Buildroot] [PATCH 0/2] Fix build of shared libraries
  2011-06-20 15:59 [Buildroot] [PATCH 0/2] Fix build of shared libraries Luca Ceresoli
  2011-06-20 15:59 ` [Buildroot] [PATCH 1/2] ffmpeg: fix " Luca Ceresoli
  2011-06-20 15:59 ` [Buildroot] [PATCH 2/2] berkeleydb: " Luca Ceresoli
@ 2011-06-21 11:29 ` Peter Korsgaard
  2 siblings, 0 replies; 4+ messages in thread
From: Peter Korsgaard @ 2011-06-21 11:29 UTC (permalink / raw)
  To: buildroot

>>>>> "Luca" == Luca Ceresoli <luca@lucaceresoli.net> writes:

 Luca> Hi,

 Luca> 009d8fceab4 and 0a5e5534cbf8 broke shared library building for
 Luca> those AUTOTOOLS packaged that override _CONFIGURE_CMDS.

 Luca> According to my grep'ing, the affetced packages should be only ffmpeg and
 Luca> berkeleydb. These two patched fixes them.

 Luca> Please note I only could test ffmpeg, but the change is pretty trivial so
 Luca> hopefully it works for berkeleydb as well.

Good catch - Committed both, thanks!

-- 
Bye, Peter Korsgaard

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

end of thread, other threads:[~2011-06-21 11:29 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-06-20 15:59 [Buildroot] [PATCH 0/2] Fix build of shared libraries Luca Ceresoli
2011-06-20 15:59 ` [Buildroot] [PATCH 1/2] ffmpeg: fix " Luca Ceresoli
2011-06-20 15:59 ` [Buildroot] [PATCH 2/2] berkeleydb: " Luca Ceresoli
2011-06-21 11:29 ` [Buildroot] [PATCH 0/2] Fix " Peter Korsgaard

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