Marc Fournier wrote:
>>>> hmmm. looks like a patch from Marc Fournier attempts to address the same
>>>> thing and I'm not sure both are necessary (and they conflict in some
>>>> cases).  I suppose it depends on what behavior is appropriate.
>>>>
>>>> Marc's patch will not perform the onlyif get/match if the node doesn't
>>>> exist (when result.nil?).  My patch will still perform the get/match; my
>>>> thinking was that this would allow to test for the entry not being
>>>> present (onlyif => "Key =~ ''").
>>>>
>>>> I'm not sure which approach results in a more intuitive behavior but
>>>> only one should be used. Thoughts?
>>>
>>> I put a test in there where, assuming no star wars characters exist in  
>>> the file this should run
>>>
>>> [...]
>> I just noticed that the extra patch got in. I reverted it. The above  
>> still holds. Marc.. does the above solve your use cases?
> 
> In fact my usual use case is
>     changes => "set Boss Nass",
>     onlyif => "get Boss != Nass ",
> as a workaround for the issue discussed in this thread:
> http://thread.gmane.org/gmane.comp.sysutils.augeas.devel/985/focus=9753
> 
> The patch I sent indeed focused on the need for this workaround. The
> behaviour of Joel's patch is definitely better.


So.. to verify... you are good?

-- bk



--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"Puppet Users" group.
To post to this group, send email to puppet-users@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-users?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to