From bc15e049826fea2ef88e2e6d1bc031835a078b77 Mon Sep 17 00:00:00 2001
From: luke <luke@980ebf18-57e1-0310-9a29-db15c13687c0>
Date: Tue, 19 Sep 2006 18:37:15 +0000
Subject: Fixing provider commands and Util#execute so they always include the
 command output when possible, as mentioned on the list

git-svn-id: https://reductivelabs.com/svn/puppet/trunk@1635 980ebf18-57e1-0310-9a29-db15c13687c0
---
 lib/puppet/provider.rb | 14 +++-----------
 lib/puppet/util.rb     |  2 +-
 2 files changed, 4 insertions(+), 12 deletions(-)

(limited to 'lib')

diff --git a/lib/puppet/provider.rb b/lib/puppet/provider.rb
index e9dcdec15..88a7ba8a6 100644
--- a/lib/puppet/provider.rb
+++ b/lib/puppet/provider.rb
@@ -51,17 +51,9 @@ class Puppet::Provider
             unless metaclass.method_defined? name
                 meta_def(name) do |args|
                     cmd = command(name) + " " + args
-                    begin
-                        output = execute cmd
-                    rescue Puppet::ExecutionFailure
-                        if output
-                            raise Puppet::ExecutionFailure.new(output)
-                        else
-                            raise Puppet::ExecutionFailure, "Could not execute '#{cmd}'"
-                        end
-                    end
-
-                    return output
+                    # This might throw an ExecutionFailure, but the system above
+                    # will catch it, if so.
+                    return execute(cmd)
                 end
                 unless method_defined? name
                     define_method(name) do |args|
diff --git a/lib/puppet/util.rb b/lib/puppet/util.rb
index e91ea0156..c6ad30e3b 100644
--- a/lib/puppet/util.rb
+++ b/lib/puppet/util.rb
@@ -436,7 +436,7 @@ module Util
 
         if failonfail
             unless $? == 0
-                raise ExecutionFailure, output
+                raise ExecutionFailure, "Could not execute '%s': %s" % [command, output]
             end
         end
 
-- 
cgit