Page MenuHomePhabricator

Survive pull/discover for hosted repositories in all VCSes
ClosedPublic

Authored by epriestley on Nov 1 2013, 10:07 PM.
Tags
None
Referenced Files
F14482129: D7474.diff
Mon, Dec 30, 7:31 AM
Unknown Object (File)
Sat, Dec 21, 5:48 AM
Unknown Object (File)
Sat, Dec 21, 4:55 AM
Unknown Object (File)
Sat, Dec 14, 2:55 PM
Unknown Object (File)
Wed, Dec 11, 3:35 PM
Unknown Object (File)
Mon, Dec 9, 9:56 PM
Unknown Object (File)
Fri, Dec 6, 7:10 AM
Unknown Object (File)
Wed, Dec 4, 12:30 PM
Subscribers

Details

Reviewers
btrahan
Maniphest Tasks
Restricted Maniphest Task
Commits
Restricted Diffusion Commit
rP3607bd487cc0: Survive pull/discover for hosted repositories in all VCSes
Summary

Hosted repositories only sometimes survive the pull/discover phases right now, due to issues like:

  • Pull tries to git clone, but should git init.
  • Mercurial doesn't handle empty repositories with on branches.
  • SVN tries to connect to an invalid remote.
  • None of them set the INIT repo flag correctly, so status doesn't get updated properly in the UI.

Fix all this stuff.

Test Plan
  • For each of Git, SVN and Mercurial:
    • Created a new repository from the web UI in a deactivated state.
    • Made it hosted.
    • Manually ran pull/discover.
    • Verified we end up with initialized, empty repositories in consistent states.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

with on branches

with "no" branches

epriestley updated this revision to Unknown Object (????).Nov 1 2013, 10:15 PM
  • "workin" -> "working"
btrahan added inline comments.
src/applications/repository/engine/PhabricatorRepositoryPullEngine.php
29

false

src/applications/repository/engine/PhabricatorRepositoryPullEngine.php
29

haha thanks

Macro bythepowerof: By the power of CODE REVIEW