change for PATCH : Do not add -lresolv on non-Linux systems

Yuping Luo Yuping.Luo at arm.com
Sun Feb 7 06:23:37 UTC 2021


Hello,

    The patch[1] define osname in Linux is "Linux", while it's "GNU/Linux"
maybe the following change is necessary to avoid build break.

yupluo01 at yupluo-ws:~/work/dev_platform/arm-linux-on-model/busybox$ git diff
diff --git a/Makefile.flags b/Makefile.flags
index 5673bc48a..01c9a3669 100644
--- a/Makefile.flags
+++ b/Makefile.flags
@@ -181,7 +181,7 @@ LDLIBS += $(if $(SELINUX_LIBS),$(SELINUX_LIBS:-l%=%),$(SELINUX_PC_MODULES:lib%=%
 endif

 ifeq ($(CONFIG_FEATURE_NSLOOKUP_BIG),y)
-ifeq ($(CONFIG_UNAME_OSNAME),Linux)
+ifneq ($(findstring Linux,$(CONFIG_UNAME_OSNAME)),)
 LDLIBS += resolv
 endif
 endif


ref:
   [1] http://lists.busybox.net/pipermail/busybox/2021-January/088426.html


Thanks,
Yuping
IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.


More information about the busybox mailing list