RapidRailsGirls / carpet-diem

MIT License
2 stars 1 forks source link

Doesn't work in Ruby 2.1.1 #16

Closed rentalcustard closed 10 years ago

rentalcustard commented 10 years ago
~> carpet_diem
/home/tom/.gem/ruby/2.1.1/gems/chingu-0.9rc9/lib/chingu/gosu_ext/sample.rb:42:in `merge!': no implicit conversion of String into Hash (TypeError)
        from /home/tom/.gem/ruby/2.1.1/gems/chingu-0.9rc9/lib/chingu/gosu_ext/sample.rb:42:in `initialize'
        from /home/tom/.gem/ruby/2.1.1/gems/carpet_diem-0.0.5/lib/lamp_with_genie.rb:16:in `new'
        from /home/tom/.gem/ruby/2.1.1/gems/carpet_diem-0.0.5/lib/lamp_with_genie.rb:16:in `initialize'
        from /home/tom/.gem/ruby/2.1.1/gems/carpet_diem-0.0.5/lib/lamp_with_genie.rb:94:in `new'
        from /home/tom/.gem/ruby/2.1.1/gems/carpet_diem-0.0.5/lib/lamp_with_genie.rb:94:in `initialize'
        from /home/tom/.gem/ruby/2.1.1/gems/carpet_diem-0.0.5/lib/window.rb:53:in `new'
        from /home/tom/.gem/ruby/2.1.1/gems/carpet_diem-0.0.5/lib/window.rb:53:in `update'
        from /home/tom/.gem/ruby/2.1.1/gems/carpet_diem-0.0.5/bin/carpet_diem:10:in `<top (required)>'
        from /home/tom/.gem/ruby/2.1.1/bin/carpet_diem:23:in `load'
        from /home/tom/.gem/ruby/2.1.1/bin/carpet_diem:23:in `<main>'

~> ruby -v
ruby 2.1.1p76 (2014-02-24 revision 45161) [x86_64-linux]
sferik commented 10 years ago

:frowning: We have been doing our development with Ruby 2.1.2. I think the problem is more likely related to Arch Linux (which we haven’t tested) than the specific Ruby version. Maybe we can find some time to test on your machine?

@RapidRailsGirls We should consider removing the chingu and texplay dependencies, since they seem to be causing more problems than they’re solving.

rentalcustard commented 10 years ago

This appears to be a different manifestation of #17 - running bin/carpet_diem inside my locally checked-out version with sound stuff removed works fine on 2.1.1

kat-crumbs commented 10 years ago

@sferik ok we removed chingu and texplay commit 9d4ba6288a0fc0c46d549dc3ee3909ded27e5d68 and also all the transparent_pixel detection methods :'(

@rentalcustard does it work now for you?

rentalcustard commented 10 years ago

Yes!