search

mrbrdo / theine

Rails preloader for JRuby (similar to Zeus, Spring and Spork)
MIT License
99 stars 10 forks source link

theine_current_ruby throws error #22

Open low-on-mana opened 8 years ago

low-on-mana commented 8 years ago

I first ran this

$ theine_set_ruby /Users/anoop/.rbenv/versions/jruby-9.0.3.0/bin/ruby
Set theine to run using /Users/anoop/.rbenv/versions/jruby-9.0.3.0/bin/ruby.

But shockingly I am getting this while using the following


$ theine_current_ruby

WARNING: Use strings for Figaro configuration. 100 was converted to "100".

NoMethodError: undefined method `include?' for nil:NilClass
  block in argv_to_s at /Users/anoop/.rbenv/versions/jruby-9.0.3.0/lib/ruby/gems/shared/gems/theine-0.0.14/lib/theine/worker.rb:105
                 map at org/jruby/RubyArray.java:2300
           argv_to_s at /Users/anoop/.rbenv/versions/jruby-9.0.3.0/lib/ruby/gems/shared/gems/theine-0.0.14/lib/theine/worker.rb:104
                 run at /Users/anoop/.rbenv/versions/jruby-9.0.3.0/lib/ruby/gems/shared/gems/theine-0.0.14/lib/theine/worker.rb:49
               <top> at /Users/anoop/.rbenv/versions/jruby-9.0.3.0/lib/ruby/gems/shared/gems/theine-0.0.14/lib/theine/worker.rb:142
rwilliams commented 4 years ago

theine_current_ruby doesn't get the current ruby, it runs theine with the current ruby. Both theine and theine_current_ruby expect some argument because it's essentially calling rails. Fixed in https://github.com/rwilliams/theine/commit/0c43e7924acabb255f405f826086c84b4dc45980