You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@buildr.apache.org by do...@apache.org on 2014/03/31 22:23:39 UTC

[3/5] git commit: Improved the error handling when generating java from wsdl in the wsgen addon.

Improved the error handling when generating java from wsdl in the wsgen addon.


Project: http://git-wip-us.apache.org/repos/asf/buildr/repo
Commit: http://git-wip-us.apache.org/repos/asf/buildr/commit/6a750095
Tree: http://git-wip-us.apache.org/repos/asf/buildr/tree/6a750095
Diff: http://git-wip-us.apache.org/repos/asf/buildr/diff/6a750095

Branch: refs/heads/master
Commit: 6a750095c659e6c00af83aefb8c3fde7b236bb27
Parents: 0b3d8a4
Author: Peter Donald <pe...@realityforge.org>
Authored: Sat Mar 29 16:04:58 2014 +1100
Committer: Peter Donald <pe...@realityforge.org>
Committed: Tue Apr 1 07:21:47 2014 +1100

----------------------------------------------------------------------
 CHANGELOG             |  2 ++
 addon/buildr/wsgen.rb | 10 ++++++++--
 2 files changed, 10 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/buildr/blob/6a750095/CHANGELOG
----------------------------------------------------------------------
diff --git a/CHANGELOG b/CHANGELOG
index 3e5fae5..d23a3e1 100644
--- a/CHANGELOG
+++ b/CHANGELOG
@@ -1,4 +1,6 @@
 1.4.16 (Pending)
+* Change: Improved the error handling when generating java from wsdl
+          in the wsgen addon.
 * Change: Upgraded the Checkstyle plugin to use checkstyle 5.7 that
           improves Java 7 compatibility.
 * Added:  Add support for configuration of the schema-pattern,

http://git-wip-us.apache.org/repos/asf/buildr/blob/6a750095/addon/buildr/wsgen.rb
----------------------------------------------------------------------
diff --git a/addon/buildr/wsgen.rb b/addon/buildr/wsgen.rb
index 1589d3d..f773f54 100644
--- a/addon/buildr/wsgen.rb
+++ b/addon/buildr/wsgen.rb
@@ -160,9 +160,15 @@ module Buildr
             end
             command << wsdl_file
 
-            `#{command.join(' ')}`
-            if $? != 0 || !File.exist?(java_file)
+            trace command.join(' ')
+            output = `#{command.join(' ')}`
+            if $? != 0
               rm_rf java_file
+              puts output
+              raise "Problem building webservices"
+            end
+            unless File.exist?(java_file)
+              puts output
               raise "Problem building webservices"
             end
           end