WIP: Add support for certctl(8) to etcupdate and mergemaster
ClosedPublic
Actions

Authored by kevans on Oct 3 2018, 4:16 AM.

Details

Reviewers

dteske
delphij
emaste
imp
allanjude

Group Reviewers

secteam

Commits

rS352950: [3/3] etcupdate and mergemaster support for certctl

Summary

This is a first draft, it needs more testing

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 22596

Event Timeline

allanjude created this revision.Oct 3 2018, 4:16 AM

Herald added a subscriber: imp. · View Herald TranscriptOct 3 2018, 4:16 AM

Harbormaster completed remote builds in B19923: Diff 48670.Oct 3 2018, 4:16 AM

I'd strongly advise adding DESTDIR support for certctl...

This revision is now accepted and ready to land.Oct 3 2018, 2:50 PM

In D17389#370970, @imp wrote:

I'd strongly advise adding DESTDIR support for certctl...

Yes, I think I will have to find a way to do that, as freebsd-update requires it as well.

kevans added a subscriber: kevans.Jan 13 2019, 5:12 AM

egypcio added a subscriber: egypcio.Jan 29 2019, 1:15 PM

kevans commandeered this revision.Feb 20 2019, 2:14 AM

kevans added a reviewer: allanjude.

Pass through DESTDIR to certctl, still needs testing

This revision now requires review to proceed.Feb 20 2019, 2:31 AM

Harbormaster completed remote builds in B22596: Diff 54109.Feb 20 2019, 2:31 AM

Backpedal, I swapped the -n clauses.

Harbormaster completed remote builds in B22598: Diff 54111.Feb 20 2019, 2:41 AM

kevans added a parent revision: D16857: Introduce certctl(8).Sep 26 2019, 5:24 PM

This revision was not accepted when it landed; it landed in state Needs Review.Oct 2 2019, 1:06 AM

Closed by commit rS352950: [3/3] etcupdate and mergemaster support for certctl (authored by kevans). · Explain Why

This revision was automatically updated to reflect the committed changes.

kevans added a commit: rS352950: [3/3] etcupdate and mergemaster support for certctl.

Revision Contents
Changeset List

Path

Size

usr.sbin/

etcupdate/

etcupdate.sh

7 lines

mergemaster/

mergemaster.sh

21 lines

Diff 54109

View Options

usr.sbin/etcupdate/etcupdate.sh