Re: [PATCH] test-lib: don't use ulimit in test prerequisites on cygwin

2017-09-14 Thread Johannes Schindelin
Hi Ramsay, On Wed, 13 Sep 2017, Ramsay Jones wrote: > So, I finally got around to testing ulimit with open files and > found that it does not limit them on cygwin (it actually fails > on the hard limit of 3200). Thank you! > Having seen Johannes email earlier, I took a chance and added MinGW to

Re: [PATCH] test-lib: don't use ulimit in test prerequisites on cygwin

2017-09-14 Thread Michael J Gruber
Jonathan Nieder venit, vidit, dixit 13.09.2017 21:20: > Ramsay Jones wrote: > >> 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 succ

Re: [PATCH] test-lib: don't use ulimit in test prerequisites on cygwin

2017-09-13 Thread Jonathan Nieder
Ramsay Jones wrote: > 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 erroneous

[PATCH] test-lib: don't use ulimit in test prerequisites on cygwin

2017-09-13 Thread Ramsay Jones
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