"GOT", but the "O" is a cute, smiling pufferfish. Index | Thread | Search

From:
Christian Weisgerber <naddy@mips.inka.de>
Subject:
Re: sparc64 regress ulimit
To:
gameoftrees@openbsd.org
Date:
Fri, 20 Jan 2023 22:01:33 +0100

Download raw body.

Thread
Tracey Emery:

> After setting up regress, it was found that 32 was too low of a ulimit
> for sparc64. The following is an idea to fix this for sparc64 and keep
> other arches untouched.

Why would sparc64 be different in this respect?  That doesn't make any
sense.

> --- regress/cmdline/checkout.sh
> +++ regress/cmdline/checkout.sh
> @@ -901,7 +901,17 @@ test_checkout_ulimit_n() {
>  	# Checkout should still work; if it does not, then either there is
>  	# a bug or the fixed limit used by this test case is no longer valid
>  	# and must be raised. Use a subshell to avoid changing global ulimit.
> -	(ulimit -n 32; got checkout -q $testroot/repo $testroot/wt \

There is absolutely no reason that limit should vary by architecture.

-- 
Christian "naddy" Weisgerber                          naddy@mips.inka.de