On Oct 19, 2010, at 1:58 PM, Jesse Wolfe wrote: > > My guess, then, is that the instances method is set to return [] because the > system requires it return something (which is probably a bug), and this way > it just returns an empty set rather than failing. > > Right, that was my understanding, too. > > It looks like the original code, before it started crashing due to the > deprecation of self[title] , was listing only the files in `/` > Is that the desired behavior?
No; I just decided that was the most rational of the not-good options. > Right now, ralsh only has two modes: 1) find one resources by title 2) find > all resources (with no parameters) > As Owen observed, actually listing all files on the system is not practical. > I think returning the empty list (or some other notion of "not implemented") > is a reasonable thing to do. I agree, but it would be nice if this worked: puppet resource / recurse=1 and produced a list of the existing files in /. -- Barondes' First Law: Science abhors contradictions; scientists' minds are replete with them. --------------------------------------------------------------------- Luke Kanies -|- http://puppetlabs.com -|- +1(615)594-8199 -- 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.
