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.

Reply via email to