Josh Triplett <josh@xxxxxxxxxxxxxxxx> writes: > This configuration option allows systematically rewriting fetch-only > URLs to push-capable URLs when used with push. For instance: > > [url "ssh://example.org/"] > pushInsteadOf = "git://example.org/" > > This will allow clones of "git://example.org/path/to/repo" to > subsequently push to "ssh://example.org/path/to/repo", without manually > configuring pushurl for that remote. Nice. > @@ -435,12 +449,13 @@ static void alias_all_urls(void) > for (i = 0; i < remotes_nr; i++) { > if (!remotes[i]) > continue; > - for (j = 0; j < remotes[i]->url_nr; j++) { > - remotes[i]->url[j] = alias_url(remotes[i]->url[j], &rewrites); > - } > for (j = 0; j < remotes[i]->pushurl_nr; j++) { > remotes[i]->pushurl[j] = alias_url(remotes[i]->pushurl[j], &rewrites); > } > + for (j = 0; j < remotes[i]->url_nr; j++) { > + remotes[i]->url[j] = alias_url(remotes[i]->url[j], &rewrites); > + add_pushurl_alias(remotes[i], remotes[i]->url[j]); > + } Even if you have URL but not pushURL, now you get a corresponding pushURL for free by just adding pushinsteadof mapping that covers the URL without having to configue pushURL for each of them. What happens if you already had a pair of concrete url and pushurl defined for one of your repositories (say git://git.kernel.org/pub/scm/git/git.git for fetch, ssh://x.kernel.org/pub/scm/git/git.git for push) at a site, and then upon seeing this new feature, added a pushinsteadof pattern that also covers the URL side of that pair (e.g. everything in git://git.kernel.org/ is mapped to x.kernel.org:/ namespsace)? Do you end up pushing to both (e.g. ssh://x.kernel.org/pub/scm/git/git.git and x.kernel.org:/pub/scm/git/git.git), or in such a case, the pushURL you gave explicitly prevents the pushinsteadof to give unexpected duplicates? -- 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