misc/97153: When -NO_KERBEROS is set, libcom_err still gets built

[Peter Losher]

>Number:         97153
>Category:       misc
>Synopsis:       When -NO_KERBEROS is set, libcom_err still gets built
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    freebsd-bugs
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Thu May 11 22:20:17 GMT 2006
>Closed-Date:
>Last-Modified:
>Originator:     Peter Losher
>Release:        6.1-RELEASE
>Organization:
ISC
>Environment:
FreeBSD foo.isc.org 6.1-RELEASE FreeBSD 6.1-RELEASE #2: Mon May  8 13:39:37 PDT 2006     plosher at foo.isc.org:/usr/obj/usr/src/sys/GENERIC  amd64

>Description:
For various reasons, the organization that work prefers to use MIT Kerberos, so normally we add NO_KERBEROS to our /etc/make.conf to prevent buildworld from building and installing the integrated Hemidal bits.  While for the most part NO_KERBEROS does it's job, it still builds libcom_err, which causes confusion when Heimdal's version gets called in the libconfig path before MIT's version does.
>How-To-Repeat:
Add NO_KERBEROS=yes to /etc/make.conf, make buildworld; make installworld.

After that check the timestamps for /usr/lib/libcom_*
>Fix:
--- lib/Makefile.orig   Thu May 11 13:42:36 2006
+++ lib/Makefile        Thu May 11 14:46:29 2006
@@ -22,7 +22,7 @@
 #
 # Otherwise, the SUBDIR list should be in alphabetical order.
 
-SUBDIR=        ${_csu} libcom_err libcrypt libkvm msun libmd libncurses \
+SUBDIR=        ${_csu} ${_libcom_err} libcrypt libkvm msun libmd libncurses \
        libnetgraph libradius librpcsvc libsbuf libtacplus libutil \
        ${_libypclnt} libalias libarchive ${_libatm} \
        libbegemot ${_libbluetooth} libbsnmp libbz2 libc ${_libc_r} \
@@ -55,6 +55,10 @@
 .if !defined(NO_BLUETOOTH)
 _libbluetooth= libbluetooth
 _libsdp=       libsdp
+.endif
+
+.if !defined(NO_KERBEROS)
+_libcom_err=   libcom_err
 .endif
 
 .if ${MACHINE_ARCH} == "i386"

>Release-Note:
>Audit-Trail:
>Unformatted: