Jeff King <peff@xxxxxxxx> writes: > On Fri, May 25, 2012 at 09:47:18PM +0200, NGUYEN Huynh Khoi Nguyen wrote: > >> git will store its configuration in ~/.config/git/config file if this file >> exists and ~/.gitconfig file doesn't, otherwise git store its configuration >> in ~/.gitconfig as usual > > What about reading? For maximum compatibility, we should always read > from _both_ of them, and choose between them only when writing, no? It > looks like your patch will only read from one or the other. > > At first people will have only one or the other, but people using > multiple versions of git, or people following already-written > instructions on the web about modifying ~/.gitconfig could end up with > both. Isn't it actually much worse than that? If you read from .gitconfig and also from the new location, but update only the new location, people who use two versions of git will be in a very confusing situation. Randomly, some of their updates are always in effect, and others only appear sometimes, and after wasting a lot of time and hair scratching their heads, they will realize that writing with old versions of Git will store values to a place visible to both versions, while writing with new versions will store values to a place visible only to new versions. I'd rather see it ignore the new location as long as ~/.gitconfig exists (and if only the new location exists, read from and write to it), and have users make a conscious decision to transition. That is: - If ~/.gitconfig exists, do not do anything new. Just exercise the original code. For these users, ~/.config/ does _not_ exist as far as Git is concerned. - (optional) If ~/.gitconfig exists, offer _moving_ it to the new location after telling the user to make sure that the user will never use older version of git again, and move it if the user agrees. - Otherwise, read from and write to the new location. -- 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