From 21dac1deee58df80f7b2cd17d661864c8db5d28a Mon Sep 17 00:00:00 2001 From: Ramsay Jones Date: Thu, 14 Sep 2017 18:24:41 +0100 Subject: test-lib: don't use ulimit in test prerequisites on cygwin On cygwin (and MinGW), the 'ulimit' built-in bash command does not have the desired effect of limiting the resources of new processes, at least for the stack and file descriptors. However, it always returns success and leads to several test prerequisites being erroneously set to true. Add a check for cygwin and MinGW to the prerequisite expressions, using a 'test_have_prereq !MINGW,!CYGWIN' clause, to guard against using ulimit. This affects the prerequisite expressions for the ULIMIT_STACK_SIZE, CMDLINE_LIMIT and ULIMIT_FILE_DESCRIPTORS prerequisites. Signed-off-by: Ramsay Jones Reviewed-by: Jonathan Nieder Signed-off-by: Junio C Hamano --- t/t6120-describe.sh | 1 - 1 file changed, 1 deletion(-) (limited to 't/t6120-describe.sh') diff --git a/t/t6120-describe.sh b/t/t6120-describe.sh index dd6dd9df9b..3d45dc2955 100755 --- a/t/t6120-describe.sh +++ b/t/t6120-describe.sh @@ -279,7 +279,6 @@ test_expect_success 'describe ignoring a borken submodule' ' grep broken out ' -# we require ulimit, this excludes Windows test_expect_failure ULIMIT_STACK_SIZE 'name-rev works in a deep repo' ' i=1 && while test $i -lt 8000 -- cgit v1.2.3