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
F14067464: D7474.diff
Tue, Nov 19, 3:09 PM
F14047040: D7474.id.diff
Thu, Nov 14, 1:33 AM
F14046636: D7474.id.diff
Wed, Nov 13, 10:36 PM
F14036977: D7474.id16845.diff
Sun, Nov 10, 1:13 PM
F14036678: D7474.diff
Sun, Nov 10, 11:34 AM
F14007187: D7474.diff
Tue, Oct 29, 1:19 AM
F13998926: D7474.diff
Thu, Oct 24, 11:51 AM
F13993017: D7474.id16845.diff
Tue, Oct 22, 8:06 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