Re: bash_completion outside repo
- From
Jeff King <peff@peff.net>
- Date
- Sep 11, 2009, 21:29 UTC
- Message-ID
- <20090911212938.GA26589@coredump.intra.peff.net>
- In-Reply-To
- <7vvdjpuqgx.fsf@alter.siamese.dyndns.org>
On Fri, Sep 11, 2009 at 02:22:06PM -0700, Junio C Hamano wrote:
Show 11 quoted lines
> So here is an illustration to handle _only_ a misspelled > branch.autosetupmerge. > > If you have this in your .git/config file: > > [branch] > autosetupmerge = nevver > > you cannot run "git diff" without this patch. But with this patch, only > the commands that _care_ about this misconfiguration would notice and > report.
OK, that example makes sense. But I'm a little dubious of how this scales to something like color.diff.plain. Who is responsible for checking? Do we do it at the beginning of every program which cares about diff values? If so, what is the failure mode when we forget (and I suspect we will, because it is easy for programs to call into unexpected code that is three layers deep)?
I think something like that needs to "belong" to the diff code itself. I guess in the case of "diff", we could check all diff-related config at diff setup time. But what about something used in several places, like core.quotepath?
-Peff