On Wed, Mar 31, 2010 at 12:47:09AM +0530, Ramkumar Ramachandra wrote: > > > The URL in the second argument can be an arbitrary string. It is up > > to the remote helper to assign meaning to it. > > I'm not sure this is correct. Here's an excerpt from remote.c > if (argc > 2) { > url = argv[2]; > } else { > url = remote->url[0]; > } > Notice how url is set to remote->url[0], in the `else` branch, where > the remote is an "ordinary remote" built by remote_get(). Even though > url may be any arbitrary string in the `if` branch, there'll be > problems when the program gets into the `else` branch transparently. Note that remote may have no URLs. Or the passed url may be like (pathological example): 'tls://srp-Ilari@[1067a6e3af81b5ea76d615280eee2bc3dd2e8079@tcp/ipv6~fe80::20e:a6ff:fe6f:2288%2]:7684/gitolite-admin' Or it may be something even more exotic with absolutely no recognizable URL-like structure, say hypothetical git-remote-fd, which would have passed syntax like: "3,4" Which doesn't even resemble an URL. -Ilari -- 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