On Sat, Jun 25, 2016 at 06:46:47AM +0000, Eric Wong wrote: > This matches the documentation and allows gc.autoPackLimit=1 > to maintain a single pack without attempting a repack on every > "git gc --auto" invocation. > > Signed-off-by: Eric Wong <e@xxxxxxxxx> > --- > builtin/gc.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/builtin/gc.c b/builtin/gc.c > index c583aad..332bcf7 100644 > --- a/builtin/gc.c > +++ b/builtin/gc.c > @@ -177,7 +177,7 @@ static int too_many_packs(void) > */ > cnt++; > } > - return gc_auto_pack_limit <= cnt; > + return gc_auto_pack_limit < cnt; > } Looks good, and I cannot think of any real downside. "0" is special for "do not use this limit", so you now have no way of asking to gc every time. But why would you want to? Asking for 1 pack is effectively "gc if something happened, otherwise do nothing". -Peff -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html