Page MenuHomePhabricator
Files F15394703

D20435.id48739.diff
No OneTemporary
Actions

D20435.id48739.diff
View Options

diff --git a/src/applications/repository/engine/PhabricatorRepositoryPullEngine.php b/src/applications/repository/engine/PhabricatorRepositoryPullEngine.php
--- a/src/applications/repository/engine/PhabricatorRepositoryPullEngine.php
+++ b/src/applications/repository/engine/PhabricatorRepositoryPullEngine.php
@@ -418,6 +418,35 @@
}
$this->installHook($root.$path);
+
+ // See T5963. When you "git clone" from a remote with no "master", the
+ // client warns you that it isn't sure what it should check out as an
+ // initial state:
+
+ // warning: remote HEAD refers to nonexistent ref, unable to checkout
+
+ // We can tell the client what it should check out by making "HEAD"
+ // point somewhere. However:
+ //
+ // (1) If we don't set "receive.denyDelteCurrent" to "ignore" and a user
+ // tries to delete the default branch, Git raises an error and refuses.
+ // We want to allow this; we already have sufficient protections around
+ // dangerous changes and do not need to special case the default branch.
+ //
+ // (2) A repository may have a nonexistent default branch configured.
+ // For now, we just respect configuration. This will raise a warning when
+ // users clone the repository.
+ //
+ // In any case, these changes are both advisory, so ignore any errors we
+ // may encounter.
+
+ if ($repository->isWorkingCopyBare()) {
+ $repository->execLocalCommand(
+ 'config -- receive.denyDeleteCurrent ignore');
+ $repository->execLocalCommand(
+ 'symbolic-ref HEAD %s',
+ 'refs/heads/'.$repository->getDefaultBranch());
+ }
}
private function loadGitRemoteRefs(

File Metadata

Mime Type
text/plain
Expires
Mon, Mar 17, 12:15 AM (2 w, 9 h ago)
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
7707894
Default Alt Text
D20435.id48739.diff (1 KB)

Event Timeline

Phabricator · Built by Phacility