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
F19470943: D7474.diff
Tue, Jan 6, 4:01 AM
F19467733: D7474.id16846.diff
Sat, Jan 3, 4:51 PM
F18969732: D7474.diff
Nov 15 2025, 7:01 AM
F18868709: D7474.id16851.diff
Nov 4 2025, 8:08 AM
F18853490: D7474.id.diff
Oct 31 2025, 5:05 PM
F18850683: D7474.diff
Oct 30 2025, 6:39 PM
F18842221: D7474.id16845.diff
Oct 28 2025, 2:09 PM
F18593907: D7474.id16851.diff
Sep 12 2025, 4:22 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