Issue #1792 has been updated by luke. Status changed from Unreviewed to Accepted
I'd say it should ignore nil, "", [], or absent, *except* for ensure -- if ensure is set to absent, it should print that. ---------------------------------------- Bug #1792: ralsh shouldn't report on properties that return nil http://projects.reductivelabs.com/issues/show/1792 Author: nigelk2 Status: Accepted Priority: Normal Assigned to: Category: Target version: Complexity: Unknown Affected version: 0.24.6 Keywords: Currently ralsh does this test for non-existent properties. <pre> if value == "" or value == [] </pre> (which could probably be replaced with value.empty?) but it seems like it should ignore nil at least, and possibly :absent. The question of :absent depends on what it is that ralsh is meant to do.... If you say do: <pre> ralsh file /no-such-file </pre> should it return a manifest of the form: <pre> file { "/no-such-file": ensure => absent, } </pre> ? It depends whether ralsh is intended to model non-existent objects as well or not, so I thought I'd throw this out as a question before submitting a patch. ---------------------------------------- You have received this notification because you have either subscribed to it, or are involved in it. To change your notification preferences, please click here: http://reductivelabs.com/redmine/my/account --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Puppet Bugs" 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-bugs?hl=en -~----------~----~----~----~------~----~------~--~---
