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
F113279270: D41548.diff
Fri, Mar 28, 1:47 PM
Unknown Object (File)
Sat, Mar 8, 8:14 PM
Unknown Object (File)
Thu, Mar 6, 10:56 AM
Unknown Object (File)
Feb 21 2025, 7:09 PM
Unknown Object (File)
Feb 20 2025, 10:09 PM
Unknown Object (File)
Feb 20 2025, 6:31 PM
Unknown Object (File)
Feb 5 2025, 11:00 PM
Unknown Object (File)
Feb 1 2025, 8:56 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.