Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 19 Apr 2020 14:54:04 -0500
From:      Kyle Evans <kevans@freebsd.org>
To:        Cy Schubert <cy@freebsd.org>
Cc:        src-committers <src-committers@freebsd.org>, svn-src-all <svn-src-all@freebsd.org>,  svn-src-head <svn-src-head@freebsd.org>
Subject:   Re: svn commit: r360102 - in head: libexec/rc/rc.d share/mk tools/build/mk
Message-ID:  <CACNAnaHs2pUPNeZmMUT-MvM6-d5Z3NXp5SEkW5KEKu5G7nmpfA@mail.gmail.com>
In-Reply-To: <202004191701.03JH1LFA002938@repo.freebsd.org>
References:  <202004191701.03JH1LFA002938@repo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Sun, Apr 19, 2020 at 12:01 PM Cy Schubert <cy@freebsd.org> wrote:
>
> Author: cy
> Date: Sun Apr 19 17:01:21 2020
> New Revision: 360102
> URL: https://svnweb.freebsd.org/changeset/base/360102
>
> Log:
>   Conditionally install Kerberos rc files based on MK_KERBEROS_SCRIPTS
>   instead of MK_KERBEROS. The reason for this change is some users
>   prefer to build FreeBSD WITHOUT_KERBEROS, wanting to retain the
>   Kerberos rc scripts to start/stop MIT Kerberos or Heimdal from ports.
>
>   PR:           197337
>   Reported by:  Adam McDougall <ebay at looksharp.net>
>   Reviewed by:  imp
>   Differential Revision:        https://reviews.freebsd.org/D24252
>
> [... snip ...]
> Modified: head/share/mk/src.opts.mk
> ==============================================================================
> --- head/share/mk/src.opts.mk   Sun Apr 19 17:01:17 2020        (r360101)
> +++ head/share/mk/src.opts.mk   Sun Apr 19 17:01:21 2020        (r360102)
> @@ -237,6 +237,7 @@ __DEFAULT_DEPENDENT_OPTIONS= \
>      INET \
>      INET6 \
>      KERBEROS \
> +    KERBEROS_SCRIPTS \
>      KVM \
>      NETGRAPH \
>      PAM \
>

This hunk seems to be wrong, looking at more context. It's setting up
a MK_KERBEROS_SCRIPTS_SUPPORT option... I'm  kinda guessing what you
intended is what the other commentary on this has been about --
defaulting KERBEROS_SCRIPTS to ON unless KERBEROS is OFF. Just moving
it up to the __DEFAULT_DEPENDENT_OPTIONS block doesn't seem to do the
trick, though, I guess kerberos is weird.

Thanks,

Kyle Evans



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CACNAnaHs2pUPNeZmMUT-MvM6-d5Z3NXp5SEkW5KEKu5G7nmpfA>