Issue #7138 has been updated by Jeff McCune. Status changed from In Topic Branch Pending Review to Merged - Pending Release
Merged into master as 8ee42bb. This should be released in 2.0.0. Thanks again for the contribution! -Jeff ---------------------------------------- Bug #7138: Facter use exec + cat ( or exec = hostname ) when not needed https://projects.puppetlabs.com/issues/7138#change-82246 Author: Michael Scherer Status: Merged - Pending Release Priority: Low Assignee: Category: interface Target version: 2.0.0 Keywords: Branch: https://github.com/puppetlabs/facter/pull/388 Affected Facter version: In various place in the code, facter use Facter::Util::Resolution.exec while this could be avoided, most notably when reading file with cat. I have patches that I can split if needed to use either ruby functions ( for hostname ), or File::open. -- 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 unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To post to this group, send email to [email protected]. Visit this group at http://groups.google.com/group/puppet-bugs?hl=en. For more options, visit https://groups.google.com/groups/opt_out.
