Fixed deprecated call to Launchy to avoid annoying deprecated warnings #62

Open
wants to merge 1 commit into
from

Conversation

Projects
None yet
5 participants
@jerefrer

See the title :)

@jerefrer

This comment has been minimized.

Show comment Hide comment
@jerefrer

jerefrer Aug 12, 2011

For information, the warning was saying:

WARNING: You made a call to a deprecated Launchy API. This call should be changed to 'Launchy.open( uri )'
WARNING: I think I was able to find the location that needs to be fixed. Please go look at:
WARNING:
WARNING: /home/jerefrer/.rvm/gems/ruby-1.8.7-p352/gems/webrat-0.7.3/lib/webrat/core/save_and_open_page.rb:22:in open_in_browser' WARNING: def open_in_browser(path) # :nodoc WARNING: require "launchy" WARNING: Launchy::Browser.run(path) WARNING: rescue LoadError WARNING: warn "Sorry, you need to install launchy to open pages:gem install launchy`"
WARNING:
WARNING: If this is not the case, please file a bug. Please file a bug at https://github.com/copiousfreetime/launchy/issues/new

For information, the warning was saying:

WARNING: You made a call to a deprecated Launchy API. This call should be changed to 'Launchy.open( uri )'
WARNING: I think I was able to find the location that needs to be fixed. Please go look at:
WARNING:
WARNING: /home/jerefrer/.rvm/gems/ruby-1.8.7-p352/gems/webrat-0.7.3/lib/webrat/core/save_and_open_page.rb:22:in open_in_browser' WARNING: def open_in_browser(path) # :nodoc WARNING: require "launchy" WARNING: Launchy::Browser.run(path) WARNING: rescue LoadError WARNING: warn "Sorry, you need to install launchy to open pages:gem install launchy`"
WARNING:
WARNING: If this is not the case, please file a bug. Please file a bug at https://github.com/copiousfreetime/launchy/issues/new

@rickmzp

This comment has been minimized.

Show comment Hide comment
@rickmzp

rickmzp Oct 3, 2011

+1

rickmzp commented Oct 3, 2011

+1

@timgentry

This comment has been minimized.

Show comment Hide comment
@timgentry

timgentry Feb 14, 2013

+1

+1

@Sodki

This comment has been minimized.

Show comment Hide comment
@Sodki

Sodki May 6, 2014

+1

Sodki commented May 6, 2014

+1

@codener

This comment has been minimized.

Show comment Hide comment
@codener

codener Jan 20, 2015

please merge this!

codener commented Jan 20, 2015

please merge this!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment