Page MenuHomeFreeBSD

Makefile.inc1: Fix -DNO_ROOT distributeworld certctl usage
ClosedPublic

Authored by jrtc27 on Jul 13 2022, 9:23 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Apr 26, 2:40 AM
Unknown Object (File)
Fri, Apr 26, 2:37 AM
Unknown Object (File)
Fri, Apr 26, 2:27 AM
Unknown Object (File)
Thu, Apr 25, 8:29 PM
Unknown Object (File)
Dec 23 2023, 12:23 PM
Unknown Object (File)
Dec 12 2023, 8:22 AM
Unknown Object (File)
Jul 8 2023, 2:41 PM
Unknown Object (File)
Jul 8 2023, 2:40 PM
Subscribers

Details

Summary

Currently for distributeworld we pass DESTDIR to certctl.sh as an
environment variable, which sets the default value in the script.
However, for -DNO_ROOT builds, CERTCTLFLAGS has METALOG_INSTALLFLAGS
which includes -D ${DESTDIR}, overriding the custom DESTDIR pointing at
the base dist directory.

Moreover, in order to ensure that the METALOG includes the base/ prefix
for all the files, we need to have certctl call install with -D set to
DESTDIR/DISTDIR without the /base suffix but also ensure the files get
installed to DESTDIR/DISTDIR/base.

Fix these by passing the custom DESTDIR to certctl via -D rather than in
the environment and to pass the /base suffix in the distributeworld case
via the newly-added -d option.

Finally we also need to run certctl rehash before we generate the .meta
files from the METALOG, not after, otherwise they won't include the
METALOG additions, so move the certctl rehash call.

Obtained from: CheriBSD
MFC after: 1 week

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.Jul 13 2022, 11:08 PM
kevans added a subscriber: markj.
kevans added inline comments.
Makefile.inc1
1451

I thought I had pushed the fix for this, but can you add this edit here as well, please? (Reported by: @markj iirc)

Makefile.inc1
1451

Ha, yes, can include when committing