Re: [PATCH 0/3] test-suite fixes for upcoming curl 8.18.0

From: Jeff King <peff@peff.net>
Date: Dec 19, 2025, 23:23 UTC
Message-ID: <20251219232357.GA3960837@coredump.intra.peff.net>
In-Reply-To: <0s72r344-865q-2n3q-o9q9-p701087s0n04@unkk.fr>

On Fri, Dec 19, 2025 at 09:47:37AM +0100, Daniel Stenberg wrote:

Show 10 quoted lines

> On Fri, 19 Dec 2025, Jeff King wrote:
> 
> > > [1] = https://github.com/curl/curl/commit/9941e7c95bf26f00fd87888a
> > 
> > and it fixes the first of Git's failing tests. But if we swap out the
> > space for a tab like this:
> 
> Sorry, that was just sloppy of me to not add a test and proper handling for
> that condition. Allow me to fix that in my end. A leading tab in the folding
> part should be replaced by a space.

Thanks! I ran Git's test suite against a build using your 6c7bc9871f (http: fix for unfolding line starting with TAB, 2025-12-19) and it works without the whitespace-relaxing in my third patch.

I also double-checked against the current tip of curl's master, which includes 3388afd2b6 (http: more unfold fixing, 2025-12-19), and everything remains fine. Thanks for a prompt fix.

Junio: I think we could just drop the third patch here, if we don't mind
test failures against an unreleased version of curl. It's in debian
unstable now, but presumably they'll move to the released 8.18.0 once
it's out.

-Peff