Skip to content

Commit c00dddc

Browse files
committed
Gracefully fail if required commands are not found, fixes issue #8
1 parent 639e738 commit c00dddc

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

src/Umpirsky/PermissionsHandler/ScriptHandler.php

+5-4
Original file line numberDiff line numberDiff line change
@@ -10,27 +10,28 @@ class ScriptHandler
1010
public static function setPermissions(CommandEvent $event)
1111
{
1212
if ('WIN' === strtoupper(substr(PHP_OS, 0, 3))) {
13-
$event->getIO()->write('No permissions setup is required on Windows.');
13+
$event->getIO()->write('<info>No permissions setup is required on Windows.</info>');
1414
return;
1515
}
16-
16+
1717
$event->getIO()->write('Setting up permissions.');
1818

1919
try {
2020
self::setPermissionsSetfacl($event);
2121

2222
return;
2323
} catch (ProcessFailedException $setfaclException) {
24+
$event->getIO()->write(sprintf('<error>%s</error>', $setfaclException->getMessage()));
25+
$event->getIO()->write('<info>Trying chmod...</info>');
2426
}
2527

2628
try {
2729
self::setPermissionsChmod($event);
2830

2931
return;
3032
} catch (ProcessFailedException $chmodException) {
33+
$event->getIO()->write(sprintf('<error>%s</error>', $chmodException->getMessage()));
3134
}
32-
33-
throw $setfaclException;
3435
}
3536

3637
public static function setPermissionsSetfacl(CommandEvent $event)

0 commit comments

Comments
 (0)