Please review pull request #52: (#12761) Update custom fact documentation. opened by (nanliu)

Description:

chomp is no longer required and the documentation is updated to reflect
the updates in Facter::Util::Resolution.

  • Opened: Tue Feb 21 21:56:05 UTC 2012
  • Based on: puppetlabs:master (26d64d43479fb59d013b3b3a7887a913c4770282)
  • Requested merge: nanliu:ticket/12761 (7676ef8626a943141b43f5ad14d0e765eeeba006)

Diff follows:

diff --git a/source/guides/custom_facts.markdown b/source/guides/custom_facts.markdown
index 53b75d8..07a24f8 100644
--- a/source/guides/custom_facts.markdown
+++ b/source/guides/custom_facts.markdown
@@ -42,11 +42,11 @@ Puppet master server:
 
     Facter.add("hardware_platform") do
       setcode do
-        Facter::Util::Resolution.exec('/bin/uname -i').chomp
+        Facter::Util::Resolution.exec('/bin/uname -i')
       end
     end
 
-Note that the `chomp` is required to provide clean data.
+Note that the `chomp` is no longer required since Facter::Util::Resolution performs it.
 
 We then use the instructions in [Plugins In Modules](./plugins_in_modules.html) page to copy
 our new fact to a module and distribute it. During your next Puppet

    

--
You received this message because you are subscribed to the Google Groups "Puppet Developers" group.
To post to this group, send email to puppet-dev@googlegroups.com.
To unsubscribe from this group, send email to puppet-dev+unsubscr...@googlegroups.com.
For more options, visit this group at http://groups.google.com/group/puppet-dev?hl=en.

Reply via email to