Re: [PATCH] t5702: be more resilient to env quoting

From: Junio C Hamano <gitster@pobox.com>
Date: Oct 31, 2026, 03:12 UTC
Message-ID: <xmqqwlqytvg9.fsf@gitster.g>
In-Reply-To: <20261031002550.GA626623@coredump.intra.peff.net>

Jeff King <peff@peff.net> writes:

> The new coreutils hit Debian unstable, so I ran into this doing my usual
> daily "make test". I wondered what idiot did the overly-complicated
> "env" invocation in the first place, but naturally it turns out it was
> me.

Namely, in eaa0fd6584 (git_connect(): fix corner cases in downgrading v2 to v0, 2023-03-17). The log message starts by discussing dropping a "push" that asks for v2 to v0 because we can only do v2 for fetches, but the three tests added by the patch verify that fetch, push, and archive invoked via a custom path do not request v2.

I can see that the patch correctly checks the value of '$GIT_PROTOCOL' in the environment when these custom commands are invoked. I would have to read the log message for the "head-scratching notes" in eaa0fd6584 if I wanted to remind myself why the original code wanted to downgrade a custom upload-pack to v0 in the first place. Luckily, I do not have to do that to be convinced that this particular patch is correct.