Issue #4433 has been updated by R.I. Pienaar.
I am stil in favor for the extension to CSV, but mostly I am in favor of making extlookup backend pluggable so we can have yaml files, csv files, your in-module data style, and whatever else looked up through the same mechanism. Your get() function has more or less done a clean room implementation of extlookup and come to the conclusion that the feature set is whats needed, so why not work on whats there already by just making it pluggable? ---------------------------------------- Feature #4433: Add hash support to extlookup https://projects.puppetlabs.com/issues/4433 Author: R.I. Pienaar Status: Needs Decision Priority: Normal Assignee: Nigel Kersten Category: functions Target version: 2.7.x Affected Puppet version: 2.6.0 Keywords: Branch: http://github.com/ripienaar/puppet/tree/extlookup_hash_support I've added hash support to extlookup. Really it's time to add yaml support but thats a whole different story. CSV files can now have: <pre> test,key=val another,key=val,foo=bar </pre> And it will return hashesh: <pre> {"key" => "val"} {"key" => "val", "foo" => "bar"} </pre> In the event that not all values have a = in them it will just behave as old and return arrays. My concern here is that we have a small case of breaking backward compatibility should someone already have key=val style values. We should make this clear in the release notes. -- 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://projects.puppetlabs.com/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.
