From 85ab7bae4eecd254388928d3a12d82d094d15ec6 Mon Sep 17 00:00:00 2001 From: Andrew Deason Date: Mon, 16 Dec 2024 23:37:40 -0600 Subject: [PATCH] cf: Disable pam for --disable-kauth Since commit f7ddab6606 (Merge pam into the kauth configure option), we set enable_pam="yes" if --enable-kauth was given to configure. That makes sense, but we also do that if --disable-kauth was given to configure. This doesn't make any sense, since we disable enable_pam and enable_kauth if no --enable-kauth/--disable-kauth option was given at all. This doesn't affect much, since INSTALL_KAUTH will be disabled for --disable-kauth, so the items in src/pam won't be installed in this case regardless. But to make our behavior more consistent, and actually avoid kauth stuff for --disable-kauth, set enable_pam to the same thing as enable_kauth when an option is given, so pam is disabled when kauth is disabled. Change-Id: Ib36e9fdffb93d659e1fadab5f1bb1334770806b8 Reviewed-on: https://gerrit.openafs.org/15995 Tested-by: BuildBot Reviewed-by: Michael Meffie Reviewed-by: Cheyenne Wills --- src/cf/options.m4 | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/cf/options.m4 b/src/cf/options.m4 index 37d773c714..f128b3630b 100644 --- a/src/cf/options.m4 +++ b/src/cf/options.m4 @@ -106,7 +106,7 @@ AC_ARG_ENABLE([kauth], [AS_HELP_STRING([--enable-kauth], [install the deprecated kauth server, pam modules, and utilities (defaults to disabled)])], - [enable_pam="yes"], + [enable_pam="$enable_kauth"], [enable_kauth="no" enable_pam="no"])