Issue #1339 has been updated by Luke Kanies. Status changed from Accepted to Duplicate
Closing this in favor of the more general #4565. ---------------------------------------- Feature #1339: Convert the Fact definition interface to rely on arguments instead of blocks http://projects.puppetlabs.com/issues/1339 Author: Luke Kanies Status: Duplicate Priority: Normal Assigned to: Category: library Target version: 2.0.0 Patch: None Keywords: Branch: The current Facter interface relies on methods called inside a block to configure the fact: <pre> Facter.add("ipaddress") do setldapname "iphostnumber" setcode { ... } end </pre> All of the configuration you can do, though, is setting attributes, so it would make more sense to switch the interface to use arguments, and then just use the block to determine the fact value: <pre> Facter.add("ipaddress", :ldapname => "iphostnumber") do ...actual code... end </pre> This would clearly require some kind of compatibility interface. It would make sense to have an interface argument that defaults to the existing interface, and then throw a warning if that interface is used: <pre> Facter.add("ipaddress", :ldapname => "iphostnumber", :api => 2) do ...actual code... end </pre> -- 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.
