You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@buildr.apache.org by dj...@apache.org on 2009/06/23 02:00:45 UTC

svn commit: r787453 - /buildr/trunk/lib/buildr/java/commands.rb

Author: djspiewak
Date: Tue Jun 23 00:00:45 2009
New Revision: 787453

URL: http://svn.apache.org/viewvc?rev=787453&view=rev
Log:
Use `sh` instead of `system` to invoke Java

Modified:
    buildr/trunk/lib/buildr/java/commands.rb

Modified: buildr/trunk/lib/buildr/java/commands.rb
URL: http://svn.apache.org/viewvc/buildr/trunk/lib/buildr/java/commands.rb?rev=787453&r1=787452&r2=787453&view=diff
==============================================================================
--- buildr/trunk/lib/buildr/java/commands.rb (original)
+++ buildr/trunk/lib/buildr/java/commands.rb Tue Jun 23 00:00:45 2009
@@ -51,9 +51,7 @@
           block = lambda { |ok, res| fail "Failed to execute #{name}, see errors above" unless ok } unless block
           puts cmd_args.join(' ') if Buildr.application.options.trace
           cmd_args = cmd_args.map(&:inspect).join(' ') if Util.win_os?
-          system(*cmd_args).tap do |ok|
-            block.call ok, $?
-          end
+          sh(*cmd_args, block)
         end
       end