On Sun, 2008-11-02 at 21:53 -0600, Luke Kanies wrote:
> On Oct 31, 2008, at 5:40 PM, Brice Figureau wrote:
> > Note:
> > * the current version only knows how to parse modules. It can't  
> > parse sole
> > manifests or manifests that are not under a standard module hierarchy.
> 
> This is a great start, though.  Are you planning on adding these? Just  
> supporting the standard site.pp in the 'manifest' setting would be  
> sufficient.

Yes, I'd like to be able to parse everything. My concern is that right
now everything must be "logically" part of a module. I can invent a
special module for the manifests that don't belong to this module
(something along a "<site>" module for instance).

> >
> > * the current version fails if you have ruby plugins (facter or  
> > other) in your
> > modules. This will be fixed soon I hope.

I think I have an idea for this, but I don't know if that will work. I
think it will be quite fragile, though as it will depend in the current
plugin API:

I think I'll parse the ruby code in a crude way, looking for strings
ala:
Facter.add("something") or newfunction(:blah, :type => :rvalue) and
backtrack to fetch the comments that are just before.
Then I'll associate those documentation objects to the module they
belong to.

> You should send a note to the user list, so the rest of the community  
> has a chance to test.

I will do as soon as it will be a little bit more stable (ie right now
if you have ruby plugins it fails).

Thanks for your comments ;-)
-- 
Brice Figureau <[EMAIL PROTECTED]>


--~--~---------~--~----~------------~-------~--~----~
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 [EMAIL PROTECTED]
For more options, visit this group at 
http://groups.google.com/group/puppet-dev?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to