On Wed, Mar 08, 2017 at 05:18:46PM +0100, Johannes Schindelin wrote: > On Wed, 8 Mar 2017, Jeff King wrote: > > > Another "non-gentle" thing I noticed here while looking at > > another thread: the repository-format version check uses the config > > parser, which will die() in certain circumstances. [...] > > Yes, that is part of the reason why I was not eager to add that check to > discover_git_directory(). The config code is die()-happy. > > This is a much bigger problem, of course, and related to a constant gripe > of mine: I *always* get the quoting wrong in my aliases. Always. And when > I want to fix it, `git config -e` simply errors out because of an invalid > config. Yes, Git, I know, that is the exact reason why I want to edit the > config in the first place. > > I am certain you will agree that this is a different topic, therefore > subject to a separate patch series. I agree that in general it is a separate issue. Technically it could cause a regression because with your series we are more eager to call discover_git_directory() even when the command would not otherwise need to even look at the current git directory. But I don't think that is the case in practice. Your new function is only called when we would try to read the config anyway. So it would only affect cases which were previously so broken they did not properly read the config (like running a command without RUN_SETUP from a subdirectory of the working tree, which was _supposed_ to respect the config but failed to do so). So yeah, I'm happy to leave it for now. -Peff