Re: [PATCHv2 1/5] update-ref: test handling large transactions properly

2015-01-22 Thread Jeff King
On Thu, Jan 22, 2015 at 11:54:49AM +0100, Michael Haggerty wrote: > > +run_with_limited_open_files () { > > + (ulimit -n 32 && "$@") > > +} > > Regarding the choice of "32", I wonder what is the worst-case number of > open file descriptors that are needed *before* counting the ones that > are c

Re: [PATCHv2 1/5] update-ref: test handling large transactions properly

2015-01-22 Thread Michael Haggerty
On 01/22/2015 03:32 AM, Stefan Beller wrote: > Signed-off-by: Stefan Beller > --- > t/t1400-update-ref.sh | 28 > 1 file changed, 28 insertions(+) > > diff --git a/t/t1400-update-ref.sh b/t/t1400-update-ref.sh > index 7b4707b..47d2fe9 100755 > --- a/t/t1400-update-re

[PATCHv2 1/5] update-ref: test handling large transactions properly

2015-01-21 Thread Stefan Beller
Signed-off-by: Stefan Beller --- t/t1400-update-ref.sh | 28 1 file changed, 28 insertions(+) diff --git a/t/t1400-update-ref.sh b/t/t1400-update-ref.sh index 7b4707b..47d2fe9 100755 --- a/t/t1400-update-ref.sh +++ b/t/t1400-update-ref.sh @@ -973,4 +973,32 @@ test_ex