search

nicksieger / sheepsafe

Keep safe from FireSheep
MIT License
211 stars 11 forks source link

Growl #3

Closed jcf closed 13 years ago

jcf commented 13 years ago

I don't know if you'll like this change but I wanted a way to prevent using Growl if it's not available instead of requiring it and logging when it's not installed properly. I think Growl's an optional dependency and tried to come up with the quickest way of using it only when present.

Is defined?(Growl) a good enough check? Let me know what you think.

Thanks for sheepsafe. Pretty cool stuff.

nicksieger commented 13 years ago

Should Growl be removed from the list of gem dependencies then? Would be nice to document that -- "If you want optional Growl notifications, sudo gem install growl."