Re: [PATCH 5/9] meson: wire up generation of distribution archive

From: Junio C Hamano <gitster@pobox.com>
Date: Jan 13, 2025, 17:55 UTC
Message-ID: <xmqqv7ui8uoh.fsf@gitster.g>
In-Reply-To: <20250113-b4-pks-meson-additions-v1-5-97f6a93f691d@pks.im>

Patrick Steinhardt <ps@pks.im> writes:

> Meson knows to generate distribution archives via `meson dist`. Despite
> generating the archive itself, this target also knows to compile and
> execute tests from that archive, which helps to ensure that the result
> is an adequate drop-in replacement for the versioned project.

My reading hiccupped at "Despite" that does not seem to say anything contradicting to what follows. Did you mean the same thing as "In addition to" there?

Show 22 quoted lines

> diff --git a/GIT-VERSION-GEN b/GIT-VERSION-GEN
> index 95d7d41d233450774a6580c0de332e7b8f69eb4e..1f0fb4098da392511f02a34cdcc84f3889771001 100755
> --- a/GIT-VERSION-GEN
> +++ b/GIT-VERSION-GEN
> @@ -19,6 +19,11 @@ then
>  	exit 1
>  fi
>  
> +if test -n "$OUTPUT" && test -n "$MESON_DIST_ROOT"
> +then
> +    OUTPUT="$MESON_DIST_ROOT/$OUTPUT"
> +fi
> +
>  DEF_VER=$(cat "$SOURCE_DIR"/GIT-VERSION)
>  
>  # Protect us from reading Git version information outside of the Git directory
> @@ -33,7 +38,7 @@ then
>  	# then try git-describe, then default.
>  	if test -f "$SOURCE_DIR"/version
>  	then
> -		VN=$(cat "$SOURCE_DIR"/version) || VN="$DEF_VER"
> +		VN=$(cat "$SOURCE_DIR"/version) && VN=${VN#GIT_VERSION=} || VN="$DEF_VER"

It used to be that the contents in the "version" file was the ultimate truth to be used as-is, but now somebody may write it with or without GIT_VERSION= prefix, and this one place is now prepared to strip the extra prefix, but everybody else who has been happily reading the "version" file is now broken until it is adjusted in the same way?