From: Tracey Emery <tracey@traceyemery.net>
Subject: sparc64 regress ulimit
To: gameoftrees@openbsd.org
Date: Thu, 19 Jan 2023 10:16:03 -0700

Hello,

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.

I aribtrarily chose 64 for the new limit, and it passed. If you want me
to test lower numbers, I can do that.

Thoughts?

-- 

Tracey Emery


diff /home/tracey/src/got
commit - 8fcd5cccdccdf61486b86753d28252df3bd5444a
path + /home/tracey/src/got
blob - 64d35a24e1ecccf1e6e9472476a2d076281f3957
file + regress/cmdline/checkout.sh
--- 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 \
+	#
+	# sparc64 requires > 32
+
+	local arch=`uname -p`;
+	if [ "$arch" = "sparc64" ]; then
+		local ul=64;
+	else
+		local ul=32;
+	fi
+
+	(ulimit -n $ul; got checkout -q $testroot/repo $testroot/wt \
 		> $testroot/stdout)
 	ret=$?
 	if [ $ret -ne 0 ]; then