Won't this add all definition and node names as classes, too?  Or was
that the previous behavior?

And shouldn't ere be some kind of test (preferably integration) for this?

-- 
http://puppetlabs.com/ | +1-615-594-8199 | @puppetmasterd

On Aug 16, 2010, at 8:16, Markus Roberts <[email protected]> wrote:

> The responsibility for adding classes to the compiler's classes list (for use
> in constructing classes.txt) moved around a bit in the 0.25 to 2.6 transition
> before being dropped in a merge conflict resolution.  Ooops.  This restores 
> it.
>
> Signed-off-by: Markus Roberts <[email protected]>
> ---
> lib/puppet/resource/type.rb |    1 +
> 1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/lib/puppet/resource/type.rb b/lib/puppet/resource/type.rb
> index 6296d26..af5740a 100644
> --- a/lib/puppet/resource/type.rb
> +++ b/lib/puppet/resource/type.rb
> @@ -69,6 +69,7 @@ class Puppet::Resource::Type
>     end
>
>     scope = subscope(scope, resource) unless resource.title == :main
> +    scope.compiler.add_class(name)
>
>     set_resource_parameters(resource, scope)
>
> --
> 1.6.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].
> 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.

Reply via email to