Re: Bug#1092497: mercurial: Please raise timeout for slower architectures and use --allow-slow-tests
On Wed, Jan 22, 2025 at 10:34:25 +0100, John Paul Adrian Glaubitz wrote:
> Hello Julien,
>
> On Wed, 2025-01-22 at 10:15 +0100, Julien Cristau wrote:
> > Hmm, so I don't think I want to do that:
> > - the default timeout is 360, so 1800 is already way above that, an hour
> > per test is unreasonable
> > - I don't want to be running the even slower tests on those already too
> > slow archs, which --allow-slow-tests would do
>
> I understand your concern. But the problem with SPARC is that the single-core
> performance is rather low and the architectures have a lot of cores to compensate
> for that. Thus, if a test requires higher single-core performance, it will fail
> here.
What --allow-slow-tests does is make some tests run that would otherwise
be skipped, so if anything it's going to make things worse.
And I'd rather skip some tests that are problematic than let them run
longer than the current timeout.
Cheers,
Julien
Reply to: