On 26 Sep 2023, at 18:32, Ravi Pokala <[email protected]> wrote: > > In that case, an updated comment seems appropriate, so it’s not removed again.
https://reviews.freebsd.org/D41984 > -Ravi (rpokala@) > > From: <[email protected]> on behalf of Warner Losh > <[email protected]> > Date: Tuesday, September 26, 2023 at 07:51 > To: Mateusz Guzik <[email protected]> > Cc: src-committers <[email protected]>, > "<[email protected]>" <[email protected]>, > "<[email protected]>" <[email protected]> > Subject: Re: git: e6dc6a27230f - main - sys/conf/newvers.sh: whack sccs tag > > Can you put this back? It is still useful. I have automated scripts that > look for this pattern to find the version of a core dump. There are no other > ways that are as reliable to find this data. Some other heuristics exist, but > my experience with snagging data from maybe a thousand core dumps from > $WORK's network found issues in all the alternatives. It's why I didn't whack > it already. Even though the text says SCCS which is prehistory for this > project, it's still in use today. > Warner > > On Tue, Sep 26, 2023, 8:37 AM Mateusz Guzik <[email protected]> wrote: >> >> >> The branch main has been updated by mjg: >> >> URL: >> https://cgit.FreeBSD.org/src/commit/?id=e6dc6a27230f836a131118326731b7c5fd417723 >> >> commit e6dc6a27230f836a131118326731b7c5fd417723 >> Author: Mateusz Guzik <[email protected]> >> AuthorDate: 2023-09-26 14:36:51 +0000 >> Commit: Mateusz Guzik <[email protected]> >> CommitDate: 2023-09-26 14:36:51 +0000 >> >> sys/conf/newvers.sh: whack sccs tag >> >> In the same spirit as removing cvs $FreeBSD$ tags. >> >> Sponsored by: Rubicon Communications, LLC ("Netgate") >> --- >> sys/conf/newvers.sh | 2 -- >> 1 file changed, 2 deletions(-) >> >> diff --git a/sys/conf/newvers.sh b/sys/conf/newvers.sh >> index 73fd6857d307..18b1817ca287 100644 >> --- a/sys/conf/newvers.sh >> +++ b/sys/conf/newvers.sh >> @@ -311,11 +311,9 @@ fi >> >> vers_content_new=$(cat << EOF >> $COPYRIGHT >> -#define SCCSSTR "@(#)${VERINFO}" >> #define VERSTR "${VERSTR}" >> #define RELSTR "${RELEASE}" >> >> -char sccs[sizeof(SCCSSTR) > 128 ? sizeof(SCCSSTR) : 128] = SCCSSTR; >> char version[sizeof(VERSTR) > 256 ? sizeof(VERSTR) : 256] = VERSTR; >> char compiler_version[] = "${compiler_v}"; >> char ostype[] = "${TYPE}";
