Page MenuHomeFreeBSD

etc: remove leftover leading empty comments, blank lines
ClosedPublic

Authored by karels on Aug 22 2023, 12:32 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Nov 6, 7:54 AM
Unknown Object (File)
Wed, Nov 6, 7:54 AM
Unknown Object (File)
Mon, Nov 4, 10:12 AM
Unknown Object (File)
Mon, Nov 4, 9:40 AM
Unknown Object (File)
Sat, Nov 2, 7:06 AM
Unknown Object (File)
Sat, Nov 2, 7:05 AM
Unknown Object (File)
Sat, Nov 2, 7:05 AM
Unknown Object (File)
Sat, Nov 2, 6:44 AM
Subscribers
None

Details

Summary

Remove leftover empty leading comments/blank lines that had been
spacers between $FreeBSD$ and the following content in config files
in src/etc.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 53235
Build 50126: arc lint + arc unit

Event Timeline

karels created this revision.

I'm not sure all of these should go... lemme check history first

Not sure about the specific ones I flagged, mostly because of trailing '#' lines.
These comments weren't in 4.4BSD and when FreeBDS added comments,
we went from a single # $Id:$ line to that, a blank # line, the comment and
a trailing # line. Weak surmise is that the leading and trailing # were intended.
But I'm not that excited either way.

The rest are fine, no question.

etc/mail/access.sample
2

I think this one should stay given the trailing #

etc/mail/mailer.conf
2

ditto

etc/mail/mailertable.sample
2

ditto

etc/mail/virtusertable.sample
2

ditto

etc/mtree/BSD.debug.dist
2

ditto for all the etc/mtree stuff

This revision is now accepted and ready to land.Aug 22 2023, 5:34 PM
etc/mtree/BSD.debug.dist
2

Does that include mtree/Makefile and mtree/README? Both had leading blank lines.

etc/mtree/BSD.debug.dist
2

Does that include mtree/Makefile and mtree/README? Both had leading blank lines.

No. Those look like they should be removed imho.