build system: remove special-casing for extra libs

It is not reliable (tried on three systems, multiple problems).

Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>

diff --git a/Config.in b/Config.in
index 11371c783d1a706848dcaec65d26115defd8cbf0..07b4bf36b18426d7e7841c618a2b1d1f236e6494 100644 (file)
--- 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 b6cd32e42327c818bebedc85801a937fcae19891..9f77674baa1966fce1e5aee5d2ae8868bad55ae1 100644 (file)
--- 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)