From c8e5ead03ac97da5b8a864481c8a790cfe8dfe4d Mon Sep 17 00:00:00 2001 From: Denys Vlasenko Date: Mon, 12 Oct 2015 03:12:17 +0200 Subject: [PATCH] build system: remove special-casing for extra libs It is not reliable (tried on three systems, multiple problems). Signed-off-by: Denys Vlasenko --- Config.in | 8 -------- Makefile.flags | 25 +++++++++++-------------- 2 files changed, 11 insertions(+), 22 deletions(-) diff --git a/Config.in b/Config.in index 11371c783..07b4bf36b 100644 --- a/Config.in +++ b/Config.in @@ -514,14 +514,6 @@ config PIE Most people will leave this set to 'N'. -config LINK_WITH_PTHREAD - bool "Link with pthread library" - default n - help - On some systems, some libraries (such as crypt) also require pthread. - - Select this only if your build otherwise fails. - config NOMMU bool "Force NOMMU build" default n diff --git a/Makefile.flags b/Makefile.flags index b6cd32e42..9f77674ba 100644 --- a/Makefile.flags +++ b/Makefile.flags @@ -129,22 +129,19 @@ else LDLIBS += m endif -ifeq ($(CONFIG_LINK_WITH_PTHREAD),y) -PTHREAD_AVAILABLE := $(shell echo 'int main(void){return 0;}' >pthreadtest.c; $(CC) $(CFLAGS) -lpthread -o /dev/null pthreadtest.c >/dev/null 2>&1 && echo "y"; rm pthreadtest.c) -ifeq ($(PTHREAD_AVAILABLE),y) -LDLIBS += pthread -endif -endif +# libpam may use libpthread, libdl and/or libaudit. +# On some platforms that requires an explicit -lpthread, -ldl, -laudit. +# However, on *other platforms* it fails when some of those flags +# given needlessly. On some systems, crypt needs pthread. +# +# I even had a system where a runtime test for pthread +# (similar to CRYPT_AVAILABLE test above) was not reliable. +# +# Do not propagate this mess by adding libraries to CONFIG_PAM/CRYPT_AVAILABLE blocks. +# Add libraries you need to CONFIG_EXTRA_LDLIBS instead. ifeq ($(CONFIG_PAM),y) -# libpam uses libpthread, libdl and libaudit, so for static builds busybox -# must be linked to libpthread, libdl and libaudit. On some platforms that -# requires an explicit -lpthread, -ldl and -laudit, so it should be in -# LDLIBS. For non-static builds, scripts/trylink will take care of removing -# these flags if possible. (Not bothering to check CONFIG_STATIC because -# even in a non-static build it could be that the only libpam available is -# libpam.a, so -lpthread & Co. could still be needed.) -LDLIBS += pam pam_misc pthread dl audit +LDLIBS += pam pam_misc endif ifeq ($(CONFIG_SELINUX),y)