Commit ac8ffddf authored by Adrian Rossouw's avatar Adrian Rossouw Committed by adrian

use drush_shell_exec where applicable, over plain exec()

parent a75e73c3
......@@ -106,8 +106,8 @@ function _provision_apache_restart_apache() {
# This is required to be configurable, due to the fact that different hosts might need to do this differently.
# TODO : add configuration / test for this
$apache_restart_cmd = escapeshellcmd(variable_get('provision_apache_restart_cmd', 'sudo apachectl graceful'));
$code = drush_shell_exec(escapeshellcmd($apache_restart_cmd));
if ($code) {
$return = drush_shell_exec(escapeshellcmd($apache_restart_cmd));
if (!$return) {
provision_set_error(PROVISION_WEB_ERROR);
provision_log("error", "Web server could not be restarted. Changes might not be available until this has been done.");
}
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment