From 764d3d1c6543dc4593986f3771ab976ce86fd66a Mon Sep 17 00:00:00 2001 From: epriestley Date: Mon, 29 Aug 2011 09:57:10 -0700 Subject: [PATCH] Make "pcntl" script error more explicit Summary: This may make it easier to debug problems with CLI + pcntl Test Plan: Added a syntax error to the script and got more useful output Reviewers: johnduhart, jungejason, tuomaspelkonen, aran Reviewed By: johnduhart CC: aran, johnduhart Differential Revision: 869 --- src/infrastructure/setup/PhabricatorSetup.php | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/infrastructure/setup/PhabricatorSetup.php b/src/infrastructure/setup/PhabricatorSetup.php index 88701f7a2c..5a27eb4e50 100644 --- a/src/infrastructure/setup/PhabricatorSetup.php +++ b/src/infrastructure/setup/PhabricatorSetup.php @@ -156,7 +156,11 @@ class PhabricatorSetup { $root); if ($err) { self::writeFailure(); - self::write("Unable to execute scripts/setup/pcntl_available.php."); + self::write("Unable to execute scripts/setup/pcntl_available.php to ". + "test for the availability of pcntl from the CLI.\n". + " err: {$err}\n". + "stdout: {$stdout}\n". + "stderr: {$stderr}\n"); return; } else { if (trim($stdout) == 'YES') {