On Fri, Jun 03, 2016 at 04:47:33PM -0700, Junio C Hamano wrote: > Mike Hommey <mh@xxxxxxxxxxxx> writes: > > > In fact, the parser doesn't even reject the one that is considered > > invalid (the first). > > My question was what the desired behaviour is, and if your "fix" > gives us that desired outcome. >From my POV, the desired outcome from this patch series is that there is no change of behavior, and Torsten's fix makes git://[example.com:123]:/path/to/repo urls handled the same before and after the patch series. Whether that's the desired behavior is another topic, that we can, IMHO, leave for later. Mike -- 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