Re: [PATCH] gc: introduce an --auto-exit-code option for undoing 3029970275

From: Jeff King <peff@peff.net>
Date: Oct 11, 2018, 00:38 UTC
Message-ID: <20181011003832.GE13853@sigill.intra.peff.net>
In-Reply-To: <87tvltecvy.fsf@evledraar.gmail.com>

On Wed, Oct 10, 2018 at 10:59:45PM +0200, Ævar Arnfjörð Bjarmason wrote:

Show 12 quoted lines

> > Callers who _are_ prepared to act on the exit code probably ought to
> > just use --auto-exit-code in their invocation.
> >
> > That said, I'm not entirely opposed to the matching config. There's
> > enough history here that somebody might want a sledgehammer setting to
> > go back to the old behavior.
> 
> If it's not a config option then as git is upgraded I'll need to change
> my across-server invocation to be some variant of checking git version,
> then etiher using the --auto-exit-code option or not (which'll error on
> older gits). Easier to be able to just drop in a config setting before
> the upgrade.

Yeah, that's the "there's enough history here" that I was referring to, but I hadn't quite thought through a concrete example. That makes sense.

(Though I also think the other part of the thread is reasonable, too, where we'd just have a command to abstract away "cat .git/gc.log" into "git gc --show-detached-log" or something).

-Peff