> > Yea, you're probably right. FWIW, here's the offending block of code
> > in lib/puppet/interface/option_builder.rb:19:
>
> [...]
> > Puppet::Interface::Option.instance_methods.grep(/=$/).each do |
> > setter|
> > next if setter =~ /^=/
> > dsl = setter.to_s.sub(/=$/, '')
>
> Yeah, that looks right to me. :)
You might want to intern it (since it was originally a symbol and the
proposed code leaves it as a string):
dsl = setter.to_s.sub(/=$/, '').intern
-- Markus
P.S. I don't think a monkey patch to add Symbol#sub is too bad either,
especially since it future-proofs the code a little bit.
--
You received this message because you are subscribed to the Google Groups
"Puppet Developers" group.
To post to this group, send email to puppet-dev@googlegroups.com.
To unsubscribe from this group, send email to
puppet-dev+unsubscr...@googlegroups.com.
For more options, visit this group at
http://groups.google.com/group/puppet-dev?hl=en.
