Browse Source

Merge pull request #141 from edpauto/failed-builds

Throw an error in case when command returns code other than 0
1.0
Andrés Montañez 10 years ago
parent
commit
03680b8271
  1. 9
      Mage/Console.php

9
Mage/Console.php

@ -120,7 +120,7 @@ class Console
} }
} }
$exceptionOccured = false;
// Run Command - Check if there is a Configuration Error // Run Command - Check if there is a Configuration Error
if ($configError !== false) { if ($configError !== false) {
self::output('<red>' . $configError . '</red>', 1, 2); self::output('<red>' . $configError . '</red>', 1, 2);
@ -136,7 +136,12 @@ class Console
} }
} }
$exitCode = $command->run(); $exitCode = $command->run();
if (is_int($exitCode) && $exitCode !== 0) {
throw new Exception("Command execution failed with following exit code: $exitCode.", $exitCode);
} elseif (is_bool($exitCode) && !$exitCode) {
$exitCode = 1;
throw new Exception("Command execution failed.", $exitCode);
}
} catch (Exception $exception) { } catch (Exception $exception) {
self::output('<red>' . $exception->getMessage() . '</red>', 1, 2); self::output('<red>' . $exception->getMessage() . '</red>', 1, 2);
} }

Loading…
Cancel
Save