Skip to content

Commit dc5ba6b

Browse files
committed
Remove MK_GSSAPI
For MIT Kerberos, MK_GSSAPI has no meaning: GSSAPI is a required part of Kerberos and is always built if MK_KERBEROS is enabled. Backport this behaviour to Heimdal so it works the same way. While here, change Heimdal's libcom_err and compile_et to be selected by MK_KERBEROS, not MK_KERBEROS_SUPPORT, since these are part of Kerberos and third-party users might need it even if Kerberos support is disabled in the base system. This means MK_KERBEROS_SUPPORT installs the same files with both MIT and Heimdal. Reviewed by: cy Differential Revision: https://reviews.freebsd.org/D51859
1 parent ba48734 commit dc5ba6b

File tree

20 files changed

+33
-46
lines changed

20 files changed

+33
-46
lines changed

Makefile.inc1

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3387,7 +3387,7 @@ secure/lib/libssh__L: lib/libz__L secure/lib/libcrypto__L lib/libcrypt__L
33873387
secure/lib/libssh__L: lib/libldns__L
33883388
.endif
33893389

3390-
.if ${MK_GSSAPI} != "no" && ${MK_KERBEROS_SUPPORT} != "no"
3390+
.if ${MK_KERBEROS_SUPPORT} != "no"
33913391
.if ${MK_MITKRB5} != "no"
33923392
secure/lib/libssh__L: krb5/lib/gssapi__L krb5/lib/krb5__L \
33933393
krb5/lib/crypto__L krb5/util/et__L lib/libmd__L krb5/util/support__L
@@ -3446,11 +3446,9 @@ kerberos5/lib/libheimipcc__L: kerberos5/lib/libroken__L kerberos5/lib/libheimbas
34463446

34473447
lib/libsqlite3__L: lib/libthr__L
34483448

3449-
.if ${MK_GSSAPI} != "no"
3450-
.if ${MK_MITKRB5} == "no"
3449+
.if ${MK_KERBEROS} != "no" && ${MK_MITKRB5} == "no"
34513450
_lib_libgssapi= lib/libgssapi
34523451
.endif
3453-
.endif
34543452

34553453
.if ${MK_KERBEROS} != "no"
34563454
.if ${MK_MITKRB5} != "no"

UPDATING

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,15 @@ NOTE TO PEOPLE WHO THINK THAT FreeBSD 15.x IS SLOW:
2727
world, or to merely disable the most expensive debugging functionality
2828
at runtime, run "ln -s 'abort:false,junk:false' /etc/malloc.conf".)
2929

30+
20250820:
31+
The WITHOUT_GSSAPI src.conf knob has been removed. This was already
32+
a no-op for MIT Kerberos, so this only affects builds which set
33+
WITHOUT_MITKRB5=1, in which case you now always get libgssapi.
34+
35+
Also, Heimdal's compile_et and libcom_err are now controlled by
36+
WITHOUT_KERBEROS, not WITHOUT_KERBEROS_SUPPORT. This matches
37+
the behaviour of the MIT Kerberos versions.
38+
3039
20250820:
3140
Commits 08c7dd2fbe4f and a4197ea47777 have changed the ABI between
3241
libvmmapi and the vmm device. If using a custom kernel configuration,

include/Makefile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -293,7 +293,7 @@ LSUBSUBDIRS+= netgraph/bluetooth/include
293293
LSUBDIRS+= fs/cuse
294294
.endif
295295

296-
.if ${MK_GSSAPI} != "no" && ${MK_MITKRB5} == "no"
296+
.if ${MK_KERBEROS} != "no" && ${MK_MITKRB5} == "no"
297297
SUBDIR+= gssapi
298298
INCS+= gssapi.h
299299
.endif

kerberos5/Makefile.inc

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -29,9 +29,7 @@ ETSRCS= \
2929
${KRB5DIR}/lib/wind/wind_err.et \
3030
${KRB5DIR}/lib/ntlm/ntlm_err.et
3131

32-
.if ${MK_GSSAPI} != "no"
3332
ETSRCS+= ${KRB5DIR}/lib/gssapi/krb5/gkrb5_err.et
34-
.endif
3533

3634
.for ET in ${ETSRCS}
3735
.for _ET in ${ET:T:R}

kerberos5/lib/Makefile

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,10 +7,8 @@ SUBDIR= libasn1 libhdb \
77
SUBDIR+= libkafs5 # requires krb_err.h from libkrb5
88
SUBDIR_DEPEND_libkafs5= libkrb5
99

10-
.if ${MK_GSSAPI} != "no"
1110
SUBDIR+= libgssapi_krb5
1211
SUBDIR+= libgssapi_ntlm
1312
SUBDIR+= libgssapi_spnego
14-
.endif
1513

1614
.include <bsd.subdir.mk>

kerberos5/libexec/Makefile

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,6 @@ SUBDIR= digest-service ipropd-master ipropd-slave hprop hpropd kdc \
44
kdigest kfd kimpersonate kpasswdd kcm
55
SUBDIR_PARALLEL=
66

7-
.if ${MK_GSSAPI} != "no"
87
SUBDIR+= kadmind
9-
.endif
108

119
.include <bsd.subdir.mk>

lib/Makefile

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -179,12 +179,12 @@ SUBDIR.${MK_FDT}+= libfdt
179179
SUBDIR.${MK_FILE}+= libmagic
180180
SUBDIR.${MK_GPIO}+= libgpio
181181
.if ${MK_MITKRB5} == "no"
182-
SUBDIR.${MK_GSSAPI}+= libgssapi
182+
SUBDIR.${MK_KERBEROS}+= libgssapi
183183
.endif
184-
SUBDIR.${MK_GSSAPI}+= librpcsec_gss
184+
SUBDIR.${MK_KERBEROS}+= librpcsec_gss
185185
SUBDIR.${MK_ICONV}+= libiconv_modules
186186
.if ${MK_MITKRB5} == "no"
187-
SUBDIR.${MK_KERBEROS_SUPPORT}+= libcom_err
187+
SUBDIR.${MK_KERBEROS}+= libcom_err
188188
.endif
189189
SUBDIR.${MK_LDNS}+= libldns
190190
SUBDIR.${MK_STATS}+= libstats

libexec/rc/rc.d/Makefile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -215,7 +215,7 @@ FTPD= ftpd
215215
FTPDPACKAGE= ftpd
216216
.endif
217217

218-
.if ${MK_GSSAPI} != "no" && ${MK_KERBEROS_SUPPORT} != "no"
218+
.if ${MK_KERBEROS_SUPPORT} != "no"
219219
CONFGROUPS+= GSSD
220220
GSSD= gssd
221221
GSSDPACKAGE= gssd

secure/libexec/sshd-session/Makefile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ LIBADD+= blacklist
3838
LDFLAGS+=-L${LIBBLACKLISTDIR}
3939
.endif
4040

41-
.if ${MK_GSSAPI} != "no" && ${MK_KERBEROS_SUPPORT} != "no"
41+
.if ${MK_KERBEROS_SUPPORT} != "no"
4242
.if ${MK_MITKRB5} != "no"
4343
LIBADD+= gssapi_krb5 krb5
4444
.include "../../krb5/Makefile.inc"

secure/ssh.mk

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,12 +9,11 @@ SKSRCS= ssh-sk-client.c
99

1010
CFLAGS+= -I${SSHDIR} -include ssh_namespace.h
1111

12-
.if ${MK_GSSAPI} != "no" && ${MK_KERBEROS_SUPPORT} != "no"
12+
.if ${MK_KERBEROS_SUPPORT} != "no"
1313
CFLAGS+= -include krb5_config.h
1414
.if ${MK_MITKRB5} == "no"
1515
CFLAGS+= -DHEIMDAL=1
1616
.endif
17-
1817
.endif
1918

2019
CFLAGS+= -DXAUTH_PATH=\"${LOCALBASE:U/usr/local}/bin/xauth\"

0 commit comments

Comments
 (0)