From: Jeff King Date: Tue, 19 Aug 2025 20:53:38 GMT Subject: Re: [PATCH 1/4] t5510: make confusing config cleanup more explicit 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: > > 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