Re: [PATCH] t4216: fix no-op test that breaks TAP output
- From
Todd Zullinger <tmz@pobox.com>
- Date
- Jun 25, 2026, 18:51 UTC
- Message-ID
- <20260625185112.jjH0K9LI@teonanacatl.net>
- In-Reply-To
- <ajjBmi39IFJW5p5V@pks.im>
Patrick Steinhardt wrote:
Show 15 quoted lines
> On Fri, Jun 19, 2026 at 09:29:44AM -0700, Junio C Hamano wrote: >> Taylor Blau <me@ttaylorr.com> writes: >> >>> Given this and the above, I would probably err on the side of >>> designating this as 'test_lazy_prereq' or otherwise silencing the output >>> of 'test_cmp' so that this does not taint the TAP output. >> >> We can argue the merit and demerit with a good log message. The >> central issue at hand is how precious 52a9 in the script lost by >> this patch is (in other words, are we checking more than "is our >> char signed or unsigned?"). > > Ultimately, I don't mind much which way we go. But if we want to retain > this, would you mind sending a rewritten v2, Taylor? I feel like you're > in a better position to argue why we should retain it.
Is this something which can be merged before 2.55.0 final? It's certainly not a grave issue, but it is a new test failure for anyone who diligently runs the test suite on many (most?) non-x86 architectures. It seems a shame to punish those folks. :)
FWIW, Tested-by: Todd Zullinger <tmz@pobox.com>
I tested the earlier test_lazy_prereq version as well.
-- Todd