Re: [PATCH 1/4] t5510: make confusing config cleanup more explicit

From: Eric Sunshine <sunshine@sunshineco.com>
Date: Aug 19, 2025, 20:16 UTC
Message-ID: <CAPig+cS5CDA_yBkHa-nuR0jBg5yygwncKAs8yK_2civBMx1gVg@mail.gmail.com>
In-Reply-To: <8797c495-8277-4f65-845b-167542b82949@charter.net>

On Tue, Aug 19, 2025 at 4:05 PM Eric Sunshine <ericsunshine@charter.net> wrote:

Show 7 quoted lines

> On 8/19/25 3:24 PM, Jeff King wrote:
> > -     test_when_finished "git config unset remote.origin.followRemoteHEAD" &&
> > +     test_when_finished "git -C \"$D/two\" config unset remote.origin.followRemoteHEAD" &&
>
> For what it's worth, I have an unsent patch from a much larger unsent
> series which cleans up the t5510 messiness differently. (The below patch
> is probably whitespace damaged by the MUA.)

For the sake of completeness regarding the "much larger unsent series": That series modifies `chainlint` to detect `cd` invocations outside of a subshell and fixes the very many instances it discovered. The series ended up consisting of 41 patches.