From: Eli Schwartz <eschwartz@gentoo.org>
Date: Mon, 13 Jan 2025 17:51:59 GMT
Subject: Re: [PATCH 2/9] GIT-VERSION-GEN: move default version into a separate file
Message-ID: <c36dc170-2792-41e2-b128-04c7c5e7ab55@gentoo.org>
In-Reply-To: <xmqqa5bua9vy.fsf@gitster.g>

On 1/13/25 12:42 PM, Junio C Hamano wrote:
> Patrick Steinhardt <ps@pks.im> writes:
> 
>>   - We can pull the default version out of GIT-VERSION-GEN and move it
>>     into its own file. This likely requires some adjustments for scripts
>>     that bump the version, but allows Meson to read the version from
>>     that file trivially.
>>
>> The last option is a proper solution and quite trivial to implement, and
>> adapting scripts should be a one-time event. Refactor GIT-VERSION-GEN
>> accordingly.
> 
> It is not clear what "proper" is.  It smells like we are bending an
> established work flow element to placate a tool that is not willing
> to cooperate, which is very much unwelcome.


If I understand correctly, the constraint is that it should work on
Windows, which means there is a bootstrap issue regarding detection of
an "sh" command for running ./GIT-VERSION-GEN

Proper simply means it works reliably on all supported targets.


> Compared to that, grepping for "^DEF_VER=" in the file may be less
> yucky.


Or for the sake of Windows portability, grep_version.py?


-- 
Eli Schwartz