Junio C Hamano <gitster@xxxxxxxxx> writes: > Daniel Barkalow <barkalow@xxxxxxxxxxxx> writes: > >>> There was a discussion that suggests that the use of colon ':' before vcs >>> helper name needs to be corrected. Nothing happened since. >> >> I believe the outcome of that discussion was: >> >> - We want to keep supporting using regular location URLs that are URLs of >> git repositories (e.g., http://git.savannah.gnu.org/cgit/xboard.git), >> and we probably want to do it with a helper which runs when >> run_command() is given "remote-<scheme>". I think installing hardlinks >> in EXECPATH ended up being the best implementation here. > > That is different from what I recall. > ... > And from my point of view, this is what is blocking the series... Just to make my position clear. I brought up the issue of "should we leave it as a single colon that confuses scp-like syntax and forces us to have three extra hardlinks?" in the message I am following up to, not because I firmly am on that side of the argument, but because I thought your version of "outcome" did not match my recollection around that particular issue. I am more or less neutral myself. Avoiding the confusion seems obviously the simple and right thing to do, and I cannot think of obvious downsides of such a change, but there probably are some downsides you have in mind, and I can well imagine the issue may become "perfect is the enemy of good". And I said "blocking the series", not in the sense that _I_ demand to change it from colon to something else, but in the sense that I recall the issue hasn't been settled in the discussion. -- 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