Re: [PATCH] ci(*-leaks): skip the git-svn tests to save time
- From
- Phillip Wood <phillip.wood123@gmail.com>
- Date
- Jan 20, 2026, 10:31 UTC
- Message-ID
- <b1ea2c89-890d-492d-9bb0-2f0f67599b2c@gmail.com>
- In-Reply-To
- <xmqq5x8z69mq.fsf@gitster.g>
On 18/01/2026 00:35, Junio C Hamano wrote:
Show 31 quoted lines
> "Kristoffer Haugsbakk" <kristofferhaugsbakk@fastmail.com> writes: > >> 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 ;-).
The dunelm address is a forwarding address that should keep working if I change my email provider. I keep meaning send a patch with a mailmap entry but never get round to actually doing it.
Thanks
Phillip