certctl: Set METALOG ownership to root:wheel
ClosedPublic
Actions

Authored by emaste on Fri, Jan 17, 9:22 PM.

Details

Reviewers

guest-patmaddox
jlduran
allanjude
jrtc27

Commits

rG4d15b58365ea: certctl: Set METALOG ownership to root:wheel

Summary

This sets the correct ownership values when building base.txz

This is an updated version of commit 10fa3f2518d4.

PR:             283340

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

This omits -N ${DESTDIR}${DISTBASE}/etc from the original commit. install does not validate the arguments passed to -o or -g (see PR283355) so there's no need to have the passwd db available.

emaste added a reviewer: jrtc27.Fri, Jan 17, 9:26 PM

Please put the rationale from your comment in the commit message, and mention that this may be improved in future to use the right DB (I intend to do that if nobody beats me to it).

This revision is now accepted and ready to land.Fri, Jan 17, 9:28 PM

Closed by commit rG4d15b58365ea: certctl: Set METALOG ownership to root:wheel (authored by guest-patmaddox, committed by emaste). · Explain WhyFri, Jan 17, 9:42 PM

This revision was automatically updated to reflect the committed changes.

emaste added a commit: rG4d15b58365ea: certctl: Set METALOG ownership to root:wheel.

Herald added a subscriber: imp. · View Herald TranscriptFri, Jan 17, 9:42 PM

emaste mentioned this in D48587: xinstall: Validate numeric uids/gids.Tue, Jan 21, 7:58 PM