Luke Kanies schrieb:
> +#   def generate
> +#       template = Puppet::Util::ResourceTemplate.new(self, 
> "/path/to/template")
> +#
> +#       return Puppet::Type.type(:file).create :path => "/my/file",
> +#           :content => template.evaluate

The functional part of the patch had this function: "def 
initialize(file, resource)", so I either my grasp of ruby is weaker than 
I feared, or you have mixed up the order of the file (path) and resource 
   ("self") param in the example.

> +# Note that it sets the generating resource's parameters, which is generally 
> most
> +# useful, since it allows you to configure the generated resource via the
> +# generating resource.

If the first "it" in this sentence refers to example generate function, 
explicitely saying so would improve the legitability. I only found this 
possible interpretation by a process of exclusion.

Regards, DavidS

--~--~---------~--~----~------------~-------~--~----~
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