[Buildroot] [PATCH] make busybox build honnor the "none" strip option

Nicolas Pitre nico at cam.org
Wed Dec 3 19:27:37 UTC 2008


The busybox build insists on doing its fancy link and strip dance,
ignoring the externally provided STRIP variable.  It however tries to
avoid stripping when -elf2flt is found in LDFLAGS or CFLAGS, but a
properly configured toolchain won't need to have -elf2flt there, and
there might be legitimate reasons to not strip busybox even on non uClinux
systems.

Fortunately there is a SKIP_STRIP variable that we can set directly.

Signed-off-by: Nicolas Pitre <nico at cam.org>

Index: package/busybox/busybox.mk
===================================================================
--- package/busybox/busybox.mk	(revision 24219)
+++ package/busybox/busybox.mk	(working copy)
@@ -109,7 +109,9 @@
 	$(MAKE) CC=$(TARGET_CC) CROSS_COMPILE="$(TARGET_CROSS)" \
 		CROSS="$(TARGET_CROSS)" PREFIX="$(TARGET_DIR)" \
 		ARCH=$(KERNEL_ARCH) \
-		EXTRA_CFLAGS="$(TARGET_CFLAGS)" -C $(BUSYBOX_DIR)
+		EXTRA_CFLAGS="$(TARGET_CFLAGS)" \
+		SKIP_STRIP=$(BR2_STRIP_none) \
+		-C $(BUSYBOX_DIR)
 ifeq ($(BR2_PREFER_IMA)$(BR2_PACKAGE_BUSYBOX_SNAPSHOT),yy)
 	rm -f $@
 	$(MAKE) CC=$(TARGET_CC) CROSS_COMPILE="$(TARGET_CROSS)" \
@@ -124,7 +126,9 @@
 	$(MAKE) CC=$(TARGET_CC) CROSS_COMPILE="$(TARGET_CROSS)" \
 		CROSS="$(TARGET_CROSS)" PREFIX="$(TARGET_DIR)" \
 		ARCH=$(KERNEL_ARCH) \
-		EXTRA_CFLAGS="$(TARGET_CFLAGS)" -C $(BUSYBOX_DIR) install
+		EXTRA_CFLAGS="$(TARGET_CFLAGS)" \
+		SKIP_STRIP=$(BR2_STRIP_none) \
+		-C $(BUSYBOX_DIR) install
 else
 	install -D -m 0755 $(BUSYBOX_DIR)/busybox $(TARGET_DIR)/bin/busybox
 endif



More information about the buildroot mailing list