Page MenuHomeFreeBSD

Don't change directory owner to root when building with -DNO_ROOT
ClosedPublic

Authored by arichardson on Feb 3 2018, 1:36 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mar 22 2024, 5:15 PM
Unknown Object (File)
Mar 22 2024, 5:15 PM
Unknown Object (File)
Mar 22 2024, 5:14 PM
Unknown Object (File)
Mar 19 2024, 7:01 AM
Unknown Object (File)
Mar 9 2024, 9:41 AM
Unknown Object (File)
Mar 3 2024, 1:23 AM
Unknown Object (File)
Dec 24 2023, 5:51 AM
Unknown Object (File)
Dec 24 2023, 5:51 AM
Subscribers
None

Details

Summary

Currently the mtree calls in Makefile.inc1 all change the directory owner
to match the spec file. However, we should not be doing this during
distributeworld if -DNO_ROOT is passed. Additionally, when creating the
WORLDTMP directory hierachy there is no need to change the owner to root so
we now always pass the -W flag when populating WORLDTMP.

This is also required for building FreeBSD on Linux/Mac since the required
groups/users will not exist there which is how I discovered this issue.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

LGTM, would like to see ideally @bdrewery but at least someone else agree

This revision is now accepted and ready to land.Feb 7 2018, 1:17 AM

Looks good to me. Thanks for splitting it out. There's no huge rush, so I'd make sure that bdrewery signs off as well.

@bdrewery does this look okay?

Yup. I am flooded with random phab reviews that are not really for me due to ports. Sorry...

This revision was automatically updated to reflect the committed changes.