From c3878390a69a39fc533346e401567227fe9190e4 Mon Sep 17 00:00:00 2001 From: "Evgeny Grin (Karlson2k)" Date: Sat, 12 Jul 2025 16:32:29 +0200 Subject: [PATCH] configure: Fix incorrect use of AM_CONDITIONAL AM_CONDITIONAL() must not be used in shell's if branches. Instead it must be specified one time only (per conditional variable) with test "something" as a second parameter. See https://www.gnu.org/software/automake/manual/html_node/Usage-of-Conditionals.html#index-AM_005fCONDITIONAL-2 Signed-off-by: Evgeny Grin (Karlson2k) Reviewed-by: Alejandro Colomar --- configure.ac | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/configure.ac b/configure.ac index 05a0407fa..da4b702f0 100644 --- a/configure.ac +++ b/configure.ac @@ -584,16 +584,15 @@ if test "$with_libpam" = "yes"; then AC_DEFINE(USE_PAM, 1, [Define to support Pluggable Authentication Modules]) AC_DEFINE_UNQUOTED(SHADOW_PAM_CONVERSATION, [$pam_conv_function],[PAM conversation to use]) - AM_CONDITIONAL(USE_PAM, [true]) AC_MSG_CHECKING(use login and su access checking if PAM not used) AC_MSG_RESULT(no) else AC_DEFINE(SU_ACCESS, 1, [Define to support /etc/suauth su access control.]) - AM_CONDITIONAL(USE_PAM, [false]) AC_MSG_CHECKING(use login and su access checking if PAM not used) AC_MSG_RESULT(yes) fi +AM_CONDITIONAL([USE_PAM], [test "X$with_libpam" = "Xyes"]) if test "$enable_acct_tools_setuid" != "no"; then if test "$with_libpam" != "yes"; then -- 2.47.2