Page MenuHomePhabricator

Distinguish between "Remote URI" and "Clone URI" in Repositories
ClosedPublic

Authored by epriestley on Jan 29 2014, 8:35 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Dec 13, 8:15 AM
Unknown Object (File)
Thu, Dec 12, 6:17 PM
Unknown Object (File)
Thu, Dec 12, 3:33 AM
Unknown Object (File)
Fri, Dec 6, 11:36 PM
Unknown Object (File)
Thu, Dec 5, 10:54 AM
Unknown Object (File)
Tue, Nov 26, 6:49 PM
Unknown Object (File)
Thu, Nov 21, 8:54 PM
Unknown Object (File)
Nov 20 2024, 5:29 AM
Subscribers
Tokens
"The World Burns" token, awarded by zeeg."Love" token, awarded by dctrwatson.

Details

Summary

Hosted repositories have muddied this distinction somewhat. In some cases, we only want to use the real remote URI, and the call is only relevant for imported repositories.

In other cases, we want the URI we'd plug into git clone.

Move this logic into PhabricatorRepository and make the distinction more clear.

Test Plan

Viewed SVN, Git, and Mercurial hosted and remote repositories, all the URIs looked reasonable.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley updated this revision to Unknown Object (????).Jan 29 2014, 8:45 PM
  • Don't strip username from hosted SSH repositories.