From 275cdfebe23f55b6fd90df55442deb9af868de55 Mon Sep 17 00:00:00 2001 From: Russ Cox Date: Mon, 24 Feb 2014 16:44:35 -0500 Subject: build: comment possible ulimit failure in run.bash Record what's going on in case someone is debugging a failure there. It's not Go's fault. Fixes issue 7381. LGTM=bradfitz R=golang-codereviews, bradfitz CC=golang-codereviews https://codereview.appspot.com/68200043 --- src/run.bash | 6 ++++++ 1 file changed, 6 insertions(+) (limited to 'src/run.bash') diff --git a/src/run.bash b/src/run.bash index 9a01e0631..a3eecff0f 100755 --- a/src/run.bash +++ b/src/run.bash @@ -17,6 +17,12 @@ ulimit -c 0 # Raise soft limits to hard limits for NetBSD/OpenBSD. # We need at least 256 files and ~300 MB of bss. # On OS X ulimit -S -n rejects 'unlimited'. +# +# Note that ulimit -S -n may fail if ulimit -H -n is set higher than a +# non-root process is allowed to set the high limit. +# This is a system misconfiguration and should be fixed on the +# broken system, not "fixed" by ignoring the failure here. +# See longer discussion on golang.org/issue/7381. [ "$(ulimit -H -n)" == "unlimited" ] || ulimit -S -n $(ulimit -H -n) [ "$(ulimit -H -d)" == "unlimited" ] || ulimit -S -d $(ulimit -H -d) -- cgit v1.2.1