From 16db51bc3c4d54bb754175009ddcae9f814109ad Mon Sep 17 00:00:00 2001 From: epriestley Date: Wed, 9 Feb 2011 09:39:17 -0800 Subject: [PATCH] Don't try to mark committed when running "arc amend --show". --- src/workflow/amend/ArcanistAmendWorkflow.php | 24 ++++++++++---------- 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/src/workflow/amend/ArcanistAmendWorkflow.php b/src/workflow/amend/ArcanistAmendWorkflow.php index 0d124da1..e8801099 100644 --- a/src/workflow/amend/ArcanistAmendWorkflow.php +++ b/src/workflow/amend/ArcanistAmendWorkflow.php @@ -107,19 +107,19 @@ EOTEXT } else { $repository_api->amendGitHeadCommit($message); echo "Amended commit message.\n"; - } - $working_copy = $this->getWorkingCopy(); - $remote_hooks = $working_copy->getConfig('remote_hooks_installed', false); - if (!$remote_hooks) { - echo "According to .arcconfig, remote commit hooks are not installed ". - "for this project, so the revision will be marked committed now. ". - "Consult the documentation for instructions on installing hooks.". - "\n\n"; - $mark_workflow = $this->buildChildWorkflow( - 'mark-committed', - array($revision_id)); - $mark_workflow->run(); + $working_copy = $this->getWorkingCopy(); + $remote_hooks = $working_copy->getConfig('remote_hooks_installed', false); + if (!$remote_hooks) { + echo "According to .arcconfig, remote commit hooks are not installed ". + "for this project, so the revision will be marked committed now. ". + "Consult the documentation for instructions on installing hooks.". + "\n\n"; + $mark_workflow = $this->buildChildWorkflow( + 'mark-committed', + array($revision_id)); + $mark_workflow->run(); + } } return 0;