danryan / spice

A zesty Chef server API wrapper
http://documentup.com/danryan/spice
MIT License
103 stars 22 forks source link

Plz namespace your Mashes correctly #28

Open goblin opened 11 years ago

goblin commented 11 years ago

I want to use the real Mash from the mash gem, together with spice. It causes unexpected, very weird behaviour and breaks horribly:

% pry
r[1] pry(main)> require 'spice'
 => true
[[2] pry(main)> require 'mash'
 => true
[[3] pry(main)> a = Mash.new('attrs' => 3)
 SystemStackError: stack level too deep
from /home/goblin/.rbenv/versions/1.9.3-p448/lib/ruby/gems/1.9.1/gems/pry-0.9.12.2/lib/pry/pry_instance.rb:328
[4] pry(main)> 

IMO your Mash should be called something like Spice::Mash