jimzah/openafs
1
0
Fork 0
mirror of https://git.openafs.org/openafs.git synced 2025-01-18 15:00:12 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

cf: Disable pam for --disable-kauth

Browse Source 
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 <buildbot@rampaginggeek.com>
Reviewed-by: Michael Meffie <mmeffie@sinenomine.net>
Reviewed-by: Cheyenne Wills <cwills@sinenomine.net>
This commit is contained in:
Andrew Deason 2024-12-16 23:37:40 -06:00
parent c4939de47d
commit 85ab7bae4e
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/cf/options.m4
View File

@ -106,7 +106,7 @@ AC_ARG_ENABLE([kauth],
[AS_HELP_STRING([--enable-kauth], [AS_HELP_STRING([--enable-kauth],
[install the deprecated kauth server, pam modules, and utilities [install the deprecated kauth server, pam modules, and utilities
(defaults to disabled)])], (defaults to disabled)])],
[enable_pam="yes"], [enable_pam="$enable_kauth"],
[enable_kauth="no" [enable_kauth="no"
enable_pam="no"]) enable_pam="no"])