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

From: Jeff King <peff@peff.net>
Date: Aug 19, 2025, 20:53 UTC
Message-ID: <20250819205338.GA1071667@coredump.intra.peff.net>
In-Reply-To: <8797c495-8277-4f65-845b-167542b82949@charter.net>

On Tue, Aug 19, 2025 at 04:03:23PM -0400, Eric Sunshine wrote:

Show 8 quoted lines

> > diff --git a/t/t5510-fetch.sh b/t/t5510-fetch.sh
> > @@ -119,7 +119,7 @@ test_expect_success "fetch test remote HEAD change" '
> > -	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.)

See my patch 2, which does this part. I split this out because it got complicated to explain why that patch wasn't breaking these lines. ;)

-Peff