Re: git: da5432eda807 - main - Remove $FreeBSD$: alt two-line .c pattern
- In reply to: Alexey Dokuchaev : "Re: git: da5432eda807 - main - Remove $FreeBSD$: alt two-line .c pattern"
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Thu, 17 Aug 2023 03:14:25 UTC
On Wed, Aug 16, 2023 at 8:59 PM Alexey Dokuchaev <danfe@freebsd.org> wrote: > On Wed, Aug 16, 2023 at 12:30:58PM -0600, Warner Losh wrote: > > On Wed, Aug 16, 2023, 12:15 PM Jessica Clarke wrote: > > > On 16 Aug 2023, at 18:57, Warner Losh wrote: > > > > commit da5432eda807c4b7232d030d5157d5b417ea4f52 > > > > > > > > Remove $FreeBSD$: alt two-line .c pattern > > > > > > > > Remove /^\s*__RCSID\("\$FreeBSD\$"\);\n\n/ > > > > ... > > > > diff --git a/sbin/routed/if.c b/sbin/routed/if.c > > > > index 83d8fe503893..72894942fe05 100644 > > > > --- a/sbin/routed/if.c > > > > +++ b/sbin/routed/if.c > > > > @@ -33,9 +33,6 @@ > > > > > > > > #include "defs.h" > > > > #include "pathnames.h" > > > > - > > > > -__RCSID("$FreeBSD$"); > > > > - > > > > struct ifhead ifnet = LIST_HEAD_INITIALIZER(ifnet); /* all > interfaces */ > > > > > > Alas these diffs aren't style(9)-compliant. > > > > Fixes welcome. I'm sure there's a hundred errors in the 32k changes > > I made. I tried to catch as many as I could... > > I think I've also noticed some triple \n\n\n as well, when #include > <cdefs.h>...__RCSID() were following top comment. Yea, I'm slowly gathering up the 'problem' patterns that I'll need to do a followup commits with. But so for they seem to be relatively small. I should have had more patterns for __RCSID. And we know about the hard-coded static char rcsid[] stuff. And the blank lines at the start of Makefiles (and a few others). There's so much variation that it's hard to capture it all (my 18 patterns weren't enough to capture all the nuance). > On a related note, > can we expect to see $FreeBSD$ back one day, when Git will be replaced > with something which does support keywords properly? > I highly doubt it. Nobody is going to put something like that back into the source tree with 32k files. We've removed them, and the current thinking is they are never coming back. But who knows what will happen in 10 or 20 years when we replace git with the next thing, so it may be unwise to say never, but I'd be greatly surprised if keywords came back into vogue. Warner