>>> 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.

Thanks !
Marc


--~--~---------~--~----~------------~-------~--~----~
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