revi-archive/phorge-arcanist
1
0
Fork 0
mirror of https://we.phorge.it/source/arcanist.git synced 2024-11-22 06:42:41 +01:00
Code Issues Releases Wiki Activity

Correctly recognize "no bookmarks" for Mercurial

Browse source 
Summary: Fixes T5086.

Test Plan: remove all bookmarks and run `arc feature`

Reviewers: epriestley, #blessed_reviewers

Subscribers: epriestley, Korvin

Maniphest Tasks: T5086

Differential Revision: https://secure.phabricator.com/D9588
This commit is contained in:
Richard van Velzen 2014-06-17 04:51:55 -07:00 committed by epriestley
parent 8d7ee9ee38
commit 3201239e85
1 changed files with 4 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/repository/api/ArcanistMercurialAPI.php
View file

@ -555,6 +555,10 @@ final class ArcanistMercurialAPI extends ArcanistRepositoryAPI {
public function getAllBranches() { public function getAllBranches() {
list($branch_info) = $this->execxLocal('bookmarks'); list($branch_info) = $this->execxLocal('bookmarks');
if (trim($branch_info) == 'no bookmarks set') {
return array();
}
$matches = null; $matches = null;
preg_match_all( preg_match_all(
'/^\s*(\*?)\s*(.+)\s(\S+)$/m', '/^\s*(\*?)\s*(.+)\s(\S+)$/m',