On Mon, May 9, 2011 at 16:11, Alex Sharp <[email protected]> wrote:

> Ruby 1.9 removed Symbol#sub, and it's used in various places
> in the puppet codebase.
>
> This is a very simple patch that converts the symbol to a
> string, performs the #sub on the string and then reconverts
> the string back to a symbol. Crude, but it should work.

We have `lib/puppet/monkey_patches.rb` to hold this sort of thing;
this would be better added there.  As an aside, it isn't entirely
clear that this is desirable behaviour, and patches to change the use
of sub on a symbol would also be accepted.  I don't have a
super-strong opinion either way, though, so if you wanted to merge
this to our monkey patch file that should be good enough to commit.

Daniel
-- 
⎋ Puppet Labs Developer – http://puppetlabs.com
✉ Daniel Pittman <[email protected]>
✆ Contact me via gtalk, email, or phone: +1 (877) 575-9775
♲ Made with 100 percent post-consumer electrons

-- 
You received this message because you are subscribed to the Google Groups 
"Puppet Developers" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/puppet-dev?hl=en.

Reply via email to