revi-archive/phorge-arcanist
1
0
Fork 0
mirror of https://we.phorge.it/source/arcanist.git synced 2024-11-10 08:52:39 +01:00
Code Issues Releases Wiki Activity

Use .^ instead of HEAD^ for mercurial

Browse source 
Summary:
##arc amend## in a mercurial repo was failing with this message

    Usage Exception: Commit 'HEAD^' is not a valid Mercurial commit identifier.

mercurial's .^ is about the same thing as git's HEAD^

Test Plan:
##arc amend## in a mercurial repo with ##"immutable_history" : "false"## in the
##.arcconfig##.

Reviewers: epriestley

Reviewed By: epriestley

CC: vrana, aran, Korvin

Differential Revision: https://secure.phabricator.com/D2690
This commit is contained in:
Jamie Wong 2012-06-08 16:57:50 -04:00
parent 08b53c3803
commit bcbe1737b8
1 changed files with 5 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
src/workflow/ArcanistAmendWorkflow.php
View file

@ -102,7 +102,11 @@ EOTEXT
$revision_id = $this->normalizeRevisionID($this->getArgument('revision'));
}
$repository_api->setRelativeCommit('HEAD^');
if ($repository_api instanceof ArcanistGitAPI) {
$repository_api->setRelativeCommit('HEAD^');
} else if ($repository_api instanceof ArcanistMercurialAPI) {
$repository_api->setRelativeCommit('.^');
}
$in_working_copy = $repository_api->loadWorkingCopyDifferentialRevisions(
$this->getConduit(),