Skip to contentSkip to conversation

util-linux: linux-pam is needed for su

PATCH New Bernd Kuhls 2 participants 2 messages Sep 3, 2017

lore
Share & tools
lore

Bernd Kuhls

author
#0 lore raw

PATCH 1/1 util-linux: linux-pam is needed for su

With linux-pam, the configure script bails out with:

configure: error: su selected, but required PAM header file not available

Fixes:
http://autobuild.buildroot.net/results/338/3381a49d17aa0d4b9711cbb36a5d7b563a8ae08f/

Signed-off-by Bernd Kuhls <[email protected]>
---
 package/util-linux/Config.in | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)
package/util-linux/Config.in +9 -1
index d2a925c1b..80aab881a 100644
@@ -330,10 +330,18 @@ config BR2_PACKAGE_UTIL_LINUX_SETTERM
 
 config BR2_PACKAGE_UTIL_LINUX_SU
 	bool "su"
-	depends on BR2_USE_MMU # fork()
+	depends on (BR2_ENABLE_LOCALE && BR2_USE_WCHAR) # linux-pam
+	depends on !BR2_STATIC_LIBS
+	depends on !BR2_TOOLCHAIN_USES_MUSL # linux-pam
+	depends on BR2_USE_MMU # fork(), linux-pam
+	select BR2_PACKAGE_LINUX_PAM
 	help
 	  Run a command with substitute user and group ID
 
+comment "su needs a uClibc or glibc toolchain w/ wchar, locale, dynamic library"
+	depends on !(BR2_ENABLE_LOCALE && BR2_USE_WCHAR) \
+		|| BR2_STATIC_LIBS || BR2_TOOLCHAIN_USES_MUSL
+
 config BR2_PACKAGE_UTIL_LINUX_SULOGIN
 	bool "sulogin"
 	depends on BR2_USE_MMU # fork()
Signature · 1 line
2.11.0
Applied
#1 lore raw

PATCH 1/1 util-linux: linux-pam is needed for su

Hello,

Quoted 10 lines · Bernd Kuhls

With linux-pam, the configure script bails out with:

configure: error: su selected, but required PAM header file not available

Fixes:
http://autobuild.buildroot.net/results/338/3381a49d17aa0d4b9711cbb36a5d7b563a8ae08f/

Signed-off-by Bernd Kuhls <[email protected]>
---
 package/util-linux/Config.in | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

Applied to master, thanks.

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com