On Fri, Sep 11, 2009 at 02:22:06PM -0700, Junio C Hamano wrote: > 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 -- 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