Date: Tue, 26 Sep 2023 10:32:19 -0700 From: Ravi Pokala <rpokala@freebsd.org> To: Warner Losh <imp@bsdimp.com>, Mateusz Guzik <mjg@freebsd.org> Cc: src-committers <src-committers@freebsd.org>, "<dev-commits-src-all@freebsd.org>" <dev-commits-src-all@freebsd.org>, "<dev-commits-src-main@freebsd.org>" <dev-commits-src-main@freebsd.org> Subject: Re: e6dc6a27230f - main - sys/conf/newvers.sh: whack sccs tag Message-ID: <52507A54-AAB7-43E9-BD06-41B9C78CF296@panasas.com> In-Reply-To: <CANCZdformxVqA0LjzOpWosgd-RUCP0r3UE=4rk9W0E6pGOGRfA@mail.gmail.com> References: <202309261437.38QEbWmw002059@gitrepo.freebsd.org> <CANCZdformxVqA0LjzOpWosgd-RUCP0r3UE=4rk9W0E6pGOGRfA@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
> This message is in MIME format. Since your mail reader does not understand this format, some or all of this message may not be legible. --B_3778569145_1222604024 Content-type: text/plain; charset="UTF-8" Content-transfer-encoding: quoted-printable In that case, an updated comment seems appropriate, so it=E2=80=99s not removed a= gain. =20 -Ravi (rpokala@) =20 =20 From: <owner-src-committers@freebsd.org> on behalf of Warner Losh <imp@bsdi= mp.com> Date: Tuesday, September 26, 2023 at 07:51 To: Mateusz Guzik <mjg@freebsd.org> Cc: src-committers <src-committers@freebsd.org>, "<dev-commits-src-all@free= bsd.org>" <dev-commits-src-all@freebsd.org>, "<dev-commits-src-main@freebsd.= org>" <dev-commits-src-main@freebsd.org> Subject: Re: git: e6dc6a27230f - main - sys/conf/newvers.sh: whack sccs tag =20 Can you put this back? It is still useful. I have automated scripts that lo= ok for this pattern to find the version of a core dump. There are no other w= ays 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 $WOR= K'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 projec= t, it's still in use today. =20 Warner =20 On Tue, Sep 26, 2023, 8:37 AM Mateusz Guzik <mjg@freebsd.org> wrote: The branch main has been updated by mjg: URL: https://cgit.FreeBSD.org/src/commit/?id=3De6dc6a27230f836a131118326731b7= c5fd417723 commit e6dc6a27230f836a131118326731b7c5fd417723 Author: Mateusz Guzik <mjg@FreeBSD.org> AuthorDate: 2023-09-26 14:36:51 +0000 Commit: Mateusz Guzik <mjg@FreeBSD.org> 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=3D$(cat << EOF $COPYRIGHT -#define SCCSSTR "@(#)${VERINFO}" #define VERSTR "${VERSTR}" #define RELSTR "${RELEASE}" -char sccs[sizeof(SCCSSTR) > 128 ? sizeof(SCCSSTR) : 128] =3D SCCSSTR; char version[sizeof(VERSTR) > 256 ? sizeof(VERSTR) : 256] =3D VERSTR; char compiler_version[] =3D "${compiler_v}"; char ostype[] =3D "${TYPE}"; --B_3778569145_1222604024 Content-type: text/html; charset="UTF-8" Content-transfer-encoding: quoted-printable <html xmlns:o=3D"urn:schemas-microsoft-com:office:office" xmlns:w=3D"urn:schema= s-microsoft-com:office:word" xmlns:m=3D"http://schemas.microsoft.com/office/20= 04/12/omml" xmlns=3D"http://www.w3.org/TR/REC-html40"><head><meta http-equiv=3DC= ontent-Type content=3D"text/html; charset=3Dutf-8"><meta name=3DGenerator content=3D= "Microsoft Word 15 (filtered medium)"><style><!-- /* Font Definitions */ @font-face {font-family:"Cambria Math"; panose-1:2 4 5 3 5 4 6 3 2 4;} @font-face {font-family:Calibri; panose-1:2 15 5 2 2 2 4 3 2 4;} @font-face {font-family:Monaco; panose-1:2 0 5 0 0 0 0 0 0 0;} @font-face {font-family:"Times New Roman \(Body CS\)"; panose-1:2 2 6 3 5 4 5 2 3 4;} /* Style Definitions */ p.MsoNormal, li.MsoNormal, div.MsoNormal {margin:0in; font-size:11.0pt; font-family:"Calibri",sans-serif;} a:link, span.MsoHyperlink {mso-style-priority:99; color:blue; text-decoration:underline;} span.EmailStyle18 {mso-style-type:personal-reply; font-family:Monaco; font-variant:normal !important; color:windowtext; text-transform:none; position:relative; top:0pt; mso-text-raise:0pt; letter-spacing:0pt; mso-contextual-alternates:no; font-weight:normal; font-style:normal; text-decoration:none none; vertical-align:baseline;} .MsoChpDefault {mso-style-type:export-only; font-size:10.0pt; mso-ligatures:none;} @page WordSection1 {size:8.5in 11.0in; margin:1.0in 1.0in 1.0in 1.0in;} div.WordSection1 {page:WordSection1;} --></style></head><body lang=3DEN-US link=3Dblue vlink=3Dpurple style=3D'word-wrap:= break-word'><div class=3DWordSection1><p class=3DMsoNormal><span style=3D'font-siz= e:10.0pt;font-family:Monaco'>In that case, an updated comment seems appropri= ate, so it=E2=80=99s not removed again.<o:p></o:p></span></p><p class=3DMsoNormal><s= pan style=3D'font-size:10.0pt;font-family:Monaco'><o:p> </o:p></span></p>= <p class=3DMsoNormal><span style=3D'font-size:10.0pt;font-family:Monaco'>-Ravi (= rpokala@)<o:p></o:p></span></p><p class=3DMsoNormal><span style=3D'font-size:10.= 0pt;font-family:Monaco'><o:p> </o:p></span></p><p class=3DMsoNormal><span= style=3D'font-size:10.0pt;font-family:Monaco'><o:p> </o:p></span></p><di= v style=3D'border:none;border-top:solid #B5C4DF 1.0pt;padding:3.0pt 0in 0in 0i= n'><p class=3DMsoNormal style=3D'margin-left:.5in'><b><span style=3D'font-size:12.= 0pt;color:black'>From: </span></b><span style=3D'font-size:12.0pt;color:black'= ><owner-src-committers@freebsd.org> on behalf of Warner Losh <imp@b= sdimp.com><br><b>Date: </b>Tuesday, September 26, 2023 at 07:51<br><b>To:= </b>Mateusz Guzik <mjg@freebsd.org><br><b>Cc: </b>src-committers <= src-committers@freebsd.org>, "<dev-commits-src-all@freebsd.org>= ;" <dev-commits-src-all@freebsd.org>, "<dev-commits-src-m= ain@freebsd.org>" <dev-commits-src-main@freebsd.org><br><b>Sub= ject: </b>Re: git: e6dc6a27230f - main - sys/conf/newvers.sh: whack sccs tag= <o:p></o:p></span></p></div><div><p class=3DMsoNormal style=3D'margin-left:.5in'= ><o:p> </o:p></p></div><div><p class=3DMsoNormal style=3D'margin-left:.5in'= >Can you put this back? It is still useful. I have automated scripts that lo= ok for this pattern to find the version of a core dump. There are no other w= ays 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 $WOR= K'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 projec= t, it's still in use today.<o:p></o:p></p><div><p class=3DMsoNormal style=3D'mar= gin-left:.5in'><o:p> </o:p></p></div><div><p class=3DMsoNormal style=3D'mar= gin-left:.5in'>Warner<o:p></o:p></p></div></div><p class=3DMsoNormal style=3D'ma= rgin-left:.5in'><o:p> </o:p></p><div><div><p class=3DMsoNormal style=3D'mar= gin-left:.5in'>On Tue, Sep 26, 2023, 8:37 AM Mateusz Guzik <<a href=3D"mail= to:mjg@freebsd.org">mjg@freebsd.org</a>> wrote:<o:p></o:p></p></div><bloc= kquote style=3D'border:none;border-left:solid #CCCCCC 1.0pt;padding:0in 0in 0i= n 6.0pt;margin-left:4.8pt;margin-right:0in'><p class=3DMsoNormal style=3D'margin= -left:.5in'>The branch main has been updated by mjg:<br><br>URL: <a href=3D"ht= tps://cgit.FreeBSD.org/src/commit/?id=3De6dc6a27230f836a131118326731b7c5fd4177= 23" target=3D"_blank">https://cgit.FreeBSD.org/src/commit/?id=3De6dc6a27230f836a= 131118326731b7c5fd417723</a><br><br>commit e6dc6a27230f836a131118326731b7c5f= d417723<br>Author: Mateusz Guzik <mjg@FreeBSD.org><= br>AuthorDate: 2023-09-26 14:36:51 +0000<br>Commit: Mateu= sz Guzik <mjg@FreeBSD.org><br>CommitDate: 2023-09-26 14:36:51 +0000<br= ><br> sys/conf/newvers.sh: whack sccs tag<br><br> = In the same spirit as removing cvs $FreeBSD$ tags.<br><br> Spon= sored by: Rubicon Communications, LLC ("Netgate")<br>-= --<br> sys/conf/newvers.sh | 2 --<br> 1 file changed, 2 deletions(= -)<br><br>diff --git a/sys/conf/newvers.sh b/sys/conf/newvers.sh<br>index 73= fd6857d307..18b1817ca287 100644<br>--- a/sys/conf/newvers.sh<br>+++ b/sys/co= nf/newvers.sh<br>@@ -311,11 +311,9 @@ fi<br><br> vers_content_new=3D$(cat= << EOF<br> $COPYRIGHT<br>-#define SCCSSTR "@(#)${VERINFO}&q= uot;<br> #define VERSTR "${VERSTR}"<br> #define RELSTR &= quot;${RELEASE}"<br><br>-char sccs[sizeof(SCCSSTR) > 128 ? sizeof(SC= CSSTR) : 128] =3D SCCSSTR;<br> char version[sizeof(VERSTR) > 256 ? siz= eof(VERSTR) : 256] =3D VERSTR;<br> char compiler_version[] =3D "${comp= iler_v}";<br> char ostype[] =3D "${TYPE}";<o:p></o:p></p><= /blockquote></div></div></body></html> --B_3778569145_1222604024--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?52507A54-AAB7-43E9-BD06-41B9C78CF296>