[Buildroot] [PATCH] portmap not building static
Thomas Petazzoni
thomas.petazzoni at free-electrons.com
Mon Sep 8 19:50:22 UTC 2014
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
More information about the buildroot
mailing list