* [Buildroot] [PATCH] portmap not building static
@ 2014-09-08 15:46 ANDY KENNEDY
2014-09-08 19:50 ` Thomas Petazzoni
0 siblings, 1 reply; 2+ messages in thread
From: ANDY KENNEDY @ 2014-09-08 15:46 UTC (permalink / raw)
To: buildroot
Portmap builds and links, however, does not get built correctly when
BR2_PREFER_STATIC_LIB is selected. There are no dynamic libraries in
rootfs.tar, however, portmap gets linked dynamically without regards to
the PREFER_STATIC_LIB tags. LDFLAGS was not being passed into the build of
portmap. Fixed.
Signed-off-by: Andy Kennedy <andy.kennedy@adtran.com>
---
diff -Naur a/package/portmap/portmap.mk b/package/portmap/portmap.mk
--- a/package/portmap/portmap.mk 2014-02-27 14:51:23.000000000 -0600
+++ b/package/portmap/portmap.mk 2014-09-08 10:38:06.264120676 -0500
@@ -15,7 +15,7 @@
endif
define PORTMAP_BUILD_CMDS
- CFLAGS="$(TARGET_CFLAGS)" \
+ CFLAGS="$(TARGET_CFLAGS)" LDFLAGS="$(TARGET_LDFLAGS)" \
$(MAKE) CC="$(TARGET_CC)" -C $(@D) $(PORTMAP_FLAGS)
endef
^ permalink raw reply [flat|nested] 2+ messages in thread
* [Buildroot] [PATCH] portmap not building static
2014-09-08 15:46 [Buildroot] [PATCH] portmap not building static ANDY KENNEDY
@ 2014-09-08 19:50 ` Thomas Petazzoni
0 siblings, 0 replies; 2+ messages in thread
From: Thomas Petazzoni @ 2014-09-08 19:50 UTC (permalink / raw)
To: buildroot
Dear ANDY KENNEDY,
The title of the patch should be:
portmap: fix static linking
On Mon, 8 Sep 2014 15:46:51 +0000, ANDY KENNEDY wrote:
> Portmap builds and links, however, does not get built correctly when
> BR2_PREFER_STATIC_LIB is selected. There are no dynamic libraries in
> rootfs.tar, however, portmap gets linked dynamically without regards to
> the PREFER_STATIC_LIB tags. LDFLAGS was not being passed into the build of
> portmap. Fixed.
Remove the "Fixed." from the commit log, this is not needed.
> diff -Naur a/package/portmap/portmap.mk b/package/portmap/portmap.mk
> --- a/package/portmap/portmap.mk 2014-02-27 14:51:23.000000000 -0600
> +++ b/package/portmap/portmap.mk 2014-09-08 10:38:06.264120676 -0500
> @@ -15,7 +15,7 @@
> endif
>
> define PORTMAP_BUILD_CMDS
> - CFLAGS="$(TARGET_CFLAGS)" \
> + CFLAGS="$(TARGET_CFLAGS)" LDFLAGS="$(TARGET_LDFLAGS)" \
> $(MAKE) CC="$(TARGET_CC)" -C $(@D) $(PORTMAP_FLAGS)
> endef
I would prefer the following fix:
diff --git a/package/portmap/portmap.mk b/package/portmap/portmap.mk
index 8203555..0189597 100644
--- a/package/portmap/portmap.mk
+++ b/package/portmap/portmap.mk
@@ -17,8 +17,7 @@ PORTMAP_FLAGS += NO_FORK=1
endif
define PORTMAP_BUILD_CMDS
- CFLAGS="$(TARGET_CFLAGS)" \
- $(MAKE) CC="$(TARGET_CC)" -C $(@D) $(PORTMAP_FLAGS)
+ $(TARGET_CONFIGURE_OPTS) $(MAKE) -C $(@D) $(PORTMAP_FLAGS)
endef
define PORTMAP_INSTALL_TARGET_CMDS
By using $(TARGET_CONFIGURE_OPTS), we ensure that all usually useful
variables are passed.
Can you resend an updated version with those changes?
Thanks!
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply related [flat|nested] 2+ messages in thread
end of thread, other threads:[~2014-09-08 19:50 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-09-08 15:46 [Buildroot] [PATCH] portmap not building static ANDY KENNEDY
2014-09-08 19:50 ` Thomas Petazzoni
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.