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)
Tue, Apr 23, 9:37 PM
Unknown Object (File)
Tue, Apr 23, 9:37 PM
Unknown Object (File)
Tue, Apr 23, 9:37 PM
Unknown Object (File)
Tue, Apr 23, 9:37 PM
Unknown Object (File)
Tue, Apr 23, 9:37 PM
Unknown Object (File)
Sat, Apr 20, 6:21 PM
Unknown Object (File)
Thu, Apr 18, 1:06 PM
Unknown Object (File)
Tue, Apr 16, 1:14 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.