Re: [PATCH] ci(*-leaks): skip the git-svn tests to save time

From: Junio C Hamano <gitster@pobox.com>
Date: Jan 18, 2026, 00:35 UTC
Message-ID: <xmqq5x8z69mq.fsf@gitster.g>
In-Reply-To: <de592bff-d7ac-41b5-bb24-4ced285a52e0@app.fastmail.com>

"Kristoffer Haugsbakk" <kristofferhaugsbakk@fastmail.com> writes:

Show 23 quoted lines

> On Sat, Jan 17, 2026, at 19:34, Junio C Hamano wrote:
>>>[snip]
>> Yup, I generally like this direction, and introducing NO_P4_TESTS
>> and NO_CVS_TESTS would not be so bad.  Here is how it looks on top
>> of Dscho's patch.
>>
>> --- >8 ---
>> Subject: [PATCH] ci: skip CVS and P4 tests in leaks job, too
>>
>> Looking at the CI logs, the p4 and cvs tests account for another 24
>> minutes of test time and they offer minimal value for quite a
>> similar reason as the previous step.
>>
>> Let's introduce and use a mechanism to skip these tests to save
>> some resources.
>>
>> Suggested-by: Phillip Wood <phillip.wood123@gmail.com>
>
> Nitpick: Using the commit ident
>
>     Phillip Wood <phillip.wood@dunelm.org.uk>
>
> might be slightly better?

I didn't even realize there are multiple addresses in play, actually. I just took it from the e-mail header's Cc: field, which my MUA copied from From: field of the message I was responding to, which was the identity of the person who suggested the change after all ;-).

So, I dunno.