[Pkg-puppet-devel] [SCM] Puppet packaging for Debian branch, experimental, updated. debian/2.6.8-1-844-g7ec39d5

Daniel Pittman daniel at puppetlabs.com
Tue May 10 08:16:36 UTC 2011


The following commit has been merged in the experimental branch:
commit 36b100a9cffdfcfe6cb661e0f41bb5cb05b19adf
Author: Daniel Pittman <daniel at puppetlabs.com>
Date:   Mon Apr 18 13:22:35 2011 -0700

    maint: print 'false' in the default render method.
    
    We used to skip printing if the return value from an action was false; this
    made it impossible to display "false" to the user, which was a more meaningful
    output in some cases than nothing.
    maint: print 'false' in the default render method.
    
    We used to skip printing if the return value from an action was false; this
    made it impossible to display "false" to the user, which was a more meaningful
    output in some cases than nothing.
    
    Reviewed-By: Max Martin <max at puppetlabs.com>

diff --git a/lib/puppet/application/face_base.rb b/lib/puppet/application/face_base.rb
index df828b5..7b5bffe 100644
--- a/lib/puppet/application/face_base.rb
+++ b/lib/puppet/application/face_base.rb
@@ -151,7 +151,7 @@ class Puppet::Application::FaceBase < Puppet::Application
     # Call the method associated with the provided action (e.g., 'find').
     if @action
       result = @face.send(@action.name, *arguments)
-      puts render(result) if result
+      puts render(result) unless result.nil?
     else
       if arguments.first.is_a? Hash
         puts "#{@face} does not have a default action"

-- 
Puppet packaging for Debian



More information about the Pkg-puppet-devel mailing list