On 14 March 2011 17:30, Daniel Pittman <dan...@puppetlabs.com> wrote: > On Mon, Mar 14, 2011 at 10:29, Paul Nasrat <pnas...@gmail.com> wrote: >> On 14 March 2011 15:46, James Turnbull <ja...@puppetlabs.com> wrote: >>> Paul Nasrat wrote: >>>> On 14 March 2011 02:57, James Turnbull <ja...@lovedthanlost.net> wrote: >>>>> Signed-off-by: James Turnbull <ja...@lovedthanlost.net> >>>> >>>> This list is kinda excessive, is there anything it doesn't work for >>>> explicitly (ie do we need the confine). >>> >>> Solaris. Hence the other command in the file. >>> >>> I don't like the list either but that's our constraint model currently. >> >> Could at least use kernel rather than operatingsystem and enumerate >> every distro name that we add. >> >> Also why can't we use default fact plus solaris specific confine? I'm >> pretty sure confined trumps non-confined - cf lib/facter/ps.rb > > They do, right now, indeed.
I believe this is how it is supposed to work, but the tests seem ambiguous see spec/unit/util/fact_spec.rb and spec/unit/util/resolution_spec.rb Paul -- 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 puppet-dev+unsubscr...@googlegroups.com. For more options, visit this group at http://groups.google.com/group/puppet-dev?hl=en.