add more operating systems support #29

Open
wants to merge 3 commits into
from

Conversation

Projects
None yet
2 participants

No description provided.

@@ -3,7 +3,7 @@ def self.menu
"
- .restart/
- conf/
- @/etc/apache2/
@trogdoro

trogdoro Feb 2, 2013

Owner

/etc/apache/ is the correct path for default OSX.

Owner

trogdoro commented Feb 2, 2013

The "/etc/apache2/" path is actually correct for the default apache on OSX. Could we dynamically show the correct path based on where httpd.conf is on the system? Something like...

    conf_dir = {
      "/usr/sbin/httpd" => "/etc/apache2/",
      "/usr/local/bin/httpd" => "/usr/local/etc/apache2/",
    }[`which httpd`.strip]   # Assume the conf dir based on where the binary is

    def self.menu
      "
      - .restart/
      - conf/
        @#{conf_dir}
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment