On Wed, Feb 3, 2016 at 10:56 AM, Jeff King <peff@xxxxxxxx> wrote: > I find it disappointing that we go back to looking for magic sequences > in the string. Could we perhaps do this more cleanly with a new config > option? Like a "user.guessIdent" which defaults to true, but people can > set to false. And without that, we do not do any automagic at all; we > get the values from the GIT_COMMITTER_* environment or the > user.{name,email} config variables, or we die(). > > I think that should allow your use case (and extend the same feature to > user.name). It wouldn't work on older versions of git, but nor would > your fix here (the only way to do that is to re-instate "(none)" as > magical). Should we generalize this use case, i.e. define a list of configuration variables that must be (re-)defined per-repo? Maybe not worth it, I don't know. I can't think of any other variable that should behave this way off the top of my head. -- Duy -- 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