Re: git: e6dc6a27230f - main - sys/conf/newvers.sh: whack sccs tag

From: Warner Losh <imp_at_bsdimp.com>
Date: Tue, 26 Sep 2023 14:51:45 UTC
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 <mjg@freebsd.org> wrote:

> The branch main has been updated by mjg:
>
> URL:
> https://cgit.FreeBSD.org/src/commit/?id=e6dc6a27230f836a131118326731b7c5fd417723
>
> 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=$(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}";
>