On Thu, Apr 23, 2009 at 08:37:26PM +0200, Johannes Sixt wrote: > > + warn("Config with no key for remote %s", name); > > + return 0; > > I don't like this. This would warn in a number of situations where it's not > obvious that remotes are involved, for example in 'git status'. Ah, I didn't think of that (I didn't really look too closely into the affected code, and you obviously have). So let me withdraw my comment. -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