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)
Thu, Apr 18, 1:06 PM
Unknown Object (File)
Tue, Apr 16, 1:14 AM
Unknown Object (File)
Tue, Apr 16, 1:14 AM
Unknown Object (File)
Tue, Apr 16, 1:13 AM
Unknown Object (File)
Tue, Apr 16, 1:06 AM
Unknown Object (File)
Tue, Apr 16, 12:50 AM
Unknown Object (File)
Mon, Apr 8, 10:28 AM
Unknown Object (File)
Thu, Apr 4, 7:26 PM
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

Repository
rP Phabricator
Branch
ruri1
Lint
Lint Passed
Unit
Tests Passed

Event Timeline

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