Page MenuHomeFreeBSD

Move master.passwd and group to lib/libc/gen/
ClosedPublic

Authored by brd on Aug 15 2018, 5:27 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Dec 10, 10:25 AM
Unknown Object (File)
Fri, Dec 6, 7:42 AM
Unknown Object (File)
Mon, Dec 2, 10:03 AM
Unknown Object (File)
Nov 10 2024, 8:34 PM
Unknown Object (File)
Nov 3 2024, 2:27 PM
Unknown Object (File)
Nov 3 2024, 6:26 AM
Unknown Object (File)
Oct 31 2024, 12:20 PM
Unknown Object (File)
Oct 24 2024, 1:21 PM
Subscribers

Details

Summary

This moves master.passwd to using CONFS= and reverts the logic back to the
way it was before, but by handling it in lib/libc/gen/Makefile.inc.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 18848
Build 18508: arc lint + arc unit

Event Timeline

It would be better to keep the master.passwd and other sensitive core file in it's place, and create a base file or base system metapackage, which pulls out the files from the proper place..

This revision is now accepted and ready to land.Aug 15 2018, 9:08 PM
This revision now requires review to proceed.Aug 16 2018, 1:21 PM

This looks okay to me for just the moving of master.passwd, though I am not found of its landing location, but can I ask for 24 hours to put together the net effect patch of what was commited, and partially reverted in combination with this patch to look for any of the pointy sticks that came up. Thanks, Rod

This revision is now accepted and ready to land.Aug 16 2018, 2:40 PM

This looks okay to me for just the moving of master.passwd, though I am not found of its landing location, but can I ask for 24 hours to put together the net effect patch of what was commited, and partially reverted in combination with this patch to look for any of the pointy sticks that came up. Thanks, Rod

Sure, I am fine to wait. I will work on other stuff.

A question has come up, isnt libc part of base, ie even in a pkgbase system libc must be installed? If that is the case then master.passwd must be installed, and I do not see a good compelling reason to move it from its current orthagonal to DESTDIR locatoin in the src tree.

A question has come up, isnt libc part of base, ie even in a pkgbase system libc must be installed? If that is the case then master.passwd must be installed, and I do not see a good compelling reason to move it from its current orthagonal to DESTDIR locatoin in the src tree.

Yes, so part of the reason is to move these to CONFS installs and the other part is to try and move things to go with the source code. I picked libc since ther are lots of functions that use master.passwd in libc.

Though I disagree with the relocation of this to libc as it is going to be installed with the absolute minimal system anyway so this delta just creates src tree churn, and user finger memory churn. You site your reason for moving it is to put it close to the sources, well, traditionally BSD sources are layed out to match the installed DESTDIR tree. I understand things like csh and sh conf files moving, that makes since in a world where csh or sh may or may not be installed by a pkgbase, however that makes no since in a world where libc and hence master.passwd shall always be installed.

How many config files are now under libc and well always be installed? Could that not of been handled by make a etc pkg that is also installed with libc?

Though I disagree with the relocation of this to libc as it is going to be installed with the absolute minimal system anyway so this delta just creates src tree churn, and user finger memory churn. You site your reason for moving it is to put it close to the sources, well, traditionally BSD sources are layed out to match the installed DESTDIR tree. I understand things like csh and sh conf files moving, that makes since in a world where csh or sh may or may not be installed by a pkgbase, however that makes no since in a world where libc and hence master.passwd shall always be installed.

How many config files are now under libc and well always be installed? Could that not of been handled by make a etc pkg that is also installed with libc?

I think the use case of not installing master.passwd is very very small.

Your last comment applies to an etc pkg as well.. And if you remove bits of the system via pkg delete that will not clean up config files with it, which I think is valuable.

Update this patch with the changes in head and include group in the same go

This revision now requires review to proceed.Oct 11 2018, 10:33 PM
brd retitled this revision from Move master.passwd to lib/libc/gen/ to Move master.passwd and group to lib/libc/gen/.Oct 11 2018, 10:36 PM
brd added reviewers: imp, gjb.

Remove a trailing semicolon that was causing build failures

This revision is now accepted and ready to land.Oct 25 2018, 1:49 PM

I thought this had already been committed?

No, @imp asked me to wait until 12.0 and I am finally getting back to working on it.

This revision was automatically updated to reflect the committed changes.