Clean up restart_command a little.

http/apache/apache_service.inc

@@ -59,8 +59,8 @@ class provisionService_http_apache extends provisionService_http {
   }
 
   function parse_configs($cause_error = FALSE) {
-    //This is required to be configurable, due to the fact that different hosts might need to do this differently.
-    //TODO : test for this instead of relying on a configuration setting?
+    // This is required to be configurable, due to the fact that different
+    // hosts might need to do this differently.
     $return = drush_shell_exec(escapeshellcmd(drush_get_option('restart_cmd')));
     if (!$return) {
       $msg = join("\n", drush_shell_exec_output());

provision.drush.inc

@@ -237,14 +237,15 @@ function drush_http_provision_named_context_save() {
 }
 
 function _provision_default_restart_cmd() {
-  $command = '/usr/sbin/apachectl'; # a proper default for most of the world
+  $command = '/usr/sbin/apachectl'; // A proper default for most of the world
   foreach (explode(':', $_SERVER['PATH']) as $path) {
     $options[] = "$path/apache2ctl";
     $options[] = "$path/apachectl";
   }
-  # try to detect the apache restart command
-  $options[] = '/usr/local/sbin/apachectl'; # freebsd
-  $options[] = '/usr/sbin/apache2ctl'; # debian + apache2
+  // Try to detect the apache restart command.
+  $options[] = '/usr/local/sbin/apachectl'; // freebsd
+  $options[] = '/usr/sbin/apache2ctl'; // debian + apache2
+  $options[] = '/usr/apache2/2.2/bin'; // Solaris
   $options[] = $command;
 
   foreach ($options as $test) {