Seems like a reasonable patch for 2.6.x to me. I verified that it fixes the problem described in the bug and doesn't break tests.
+1 On Tue, Sep 21, 2010 at 3:13 PM, Markus Roberts <[email protected]> wrote: > This is a minimal fix for #4631 by reversing one part of the refactor for > #1903 > commit 0d4fd60c7c143cc1f4e4b0f99f359c09cbfbf21e. It fixes the immediate > issue > (implicit classes not being added to resources) but leaves open several > other > questions which are defered to #4687. > > Signed-off-by: Markus Roberts <[email protected]> > --- > lib/puppet/parser/resource.rb | 7 +++++++ > 1 files changed, 7 insertions(+), 0 deletions(-) > > diff --git a/lib/puppet/parser/resource.rb b/lib/puppet/parser/resource.rb > index 8a5ae88..e34f284 100644 > --- a/lib/puppet/parser/resource.rb > +++ b/lib/puppet/parser/resource.rb > @@ -93,6 +93,7 @@ class Puppet::Parser::Resource < Puppet::Resource > @finished = true > add_defaults > add_metaparams > + add_scope_tags > validate > end > > @@ -259,6 +260,12 @@ class Puppet::Parser::Resource < Puppet::Resource > end > end > > + def add_scope_tags > + if scope_resource = scope.resource > + tag(*scope_resource.tags) > + end > + end > + > # Accept a parameter from an override. > def override_parameter(param) > # This can happen if the override is defining a new parameter, rather > -- > 1.7.0.4 > > -- > 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]<puppet-dev%[email protected]> > . > For more options, visit this group at > http://groups.google.com/group/puppet-dev?hl=en. > > -- 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.
