rc.d: Fix Kerberos rc scripts
ClosedPublic
Actions

Authored by ivy on Fri, Aug 8, 5:26 PM.

Details

Reviewers

des

Group Reviewers

krb5

Commits

rG992d2f928452: rc.d: Fix Kerberos rc scripts

Summary

Split the Kerberos rc scripts into two sections, one for MIT Kerberos
and one for Heimdal. Don't install Heimdal-specific rc scripts (e.g.,
kfd) for the MIT case.

This does not include an OptionalObsoleteFiles change (even though one
is needed) because that will be fixed later in a different way.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 66130
Build 63013: arc lint + arc unit

Event Timeline

ivy created this revision.Fri, Aug 8, 5:26 PM

Herald added a subscriber: imp. · View Herald TranscriptFri, Aug 8, 5:26 PM

ivy requested review of this revision.Fri, Aug 8, 5:26 PM

Harbormaster completed remote builds in B66118: Diff 160075.Fri, Aug 8, 5:26 PM

des accepted this revision.Fri, Aug 8, 9:32 PM

This revision is now accepted and ready to land.Fri, Aug 8, 9:32 PM

the MIT scripts should be in kerberos-kdc, and also remove some now-unused variables

This revision now requires review to proceed.Sat, Aug 9, 6:39 AM

Harbormaster completed remote builds in B66130: Diff 160105.Sat, Aug 9, 6:39 AM

des added inline comments.Sun, Aug 10, 10:27 AM

libexec/rc/rc.d/Makefile
286	This might be easier to read as `.if ${MK_MITKRB5} == "no" ... .else ... .fi` within an outer `.if ${MK_KERBEROS} != "no" ... .fi`.

This revision was not accepted when it landed; it landed in state Needs Review.Sun, Aug 10, 3:38 PM

Closed by commit rG992d2f928452: rc.d: Fix Kerberos rc scripts (authored by ivy). · Explain Why

This revision was automatically updated to reflect the committed changes.

ivy added a commit: rG992d2f928452: rc.d: Fix Kerberos rc scripts.

Revision Contents
Changeset List

Path

Size

libexec/

rc/

rc.d/

Makefile

31 lines

Diff 160105

View Options

rc.d: Fix Kerberos rc scriptsClosedPublicActions