Re: [PATCH 0/1] quote: quote space

From: Junio C Hamano <gitster@pobox.com>
Date: Mar 28, 2024, 16:19 UTC
Message-ID: <xmqq34sawcqr.fsf@gitster.g>
In-Reply-To: <20240328103254.GA898963@coredump.intra.peff.net>

Jeff King <peff@peff.net> writes:

Show 11 quoted lines

>> With the following band-aid, we can skip the test and the output
>> from "sh t4126-*.sh -i -v -x" might give us a clue that explains how
>> such a failure happens.  Unfortunately GitHub CI's win test does not
>> give us insight into a test that did not fail, so I did not get
>> anything useful from the "ls -l" down there (I already knew that
>> sample patches are empty files).
>
> We package up the failed test output and trash directories for each run.
> You can find the one for this case here:
>
>   https://github.com/git/git/actions/runs/8458842054/artifacts/1364695605

What I meant was that with the band-aid that (1) sets prerequisite so that Windows would not fail and (2) has some diagnostic in the code that sets prerequisite, because the overall test does not fail, we do not package up that diagnostic output.

> On the plus side, while trying to find the failing CI job, I ran across
> and diagnosed two other unrelated failures in "seen". ;)

Thanks.