interagent/pliny

Object::Config is not defined in MRI 2.2.0

Closed this issue · 5 comments

jnv commented

config_helpers file attempts to remove the Config constant due to a deprecation warning in earlier versions of MRI. In MRI 2.2.0 the constant is finally removed, but this breaks Pliny under 2.2.0 with NameError (constant Object::Config not defined).

The line should check whether the constant is actually defined on Object and only then remove it, e.g.:

Object.send :remove_const, :Config if Object.const_defined? :Config
jnv commented

Thank you and Happy New Year! 🎉

brandur commented

Nice catch and fix guys! Thanks.

neilmiddleton commented

Can we post a release so I can use this on 2.2.0? It doesn't appear that this is in the latest tag.

brandur commented

@neilmiddleton Yep! Pushed 0.6.0. Let us know if you run into any other issues.

/cc @will