run-jsc-stress-tests should be more paranoid about calling make
authorfpizlo@apple.com <fpizlo@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 8 Sep 2013 20:58:56 +0000 (20:58 +0000)
committerfpizlo@apple.com <fpizlo@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 8 Sep 2013 20:58:56 +0000 (20:58 +0000)
https://bugs.webkit.org/show_bug.cgi?id=121011

Reviewed by Oliver Hunt.

If it returns with any kind of error then bail out.  Also print the command being used
in case we have to debug this, for example on the bots.  The extra verbosity isn't a
big deal.

* Scripts/run-jsc-stress-tests:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@155314 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Tools/ChangeLog
Tools/Scripts/run-jsc-stress-tests

index e44c74b..b238126 100644 (file)
@@ -1,5 +1,18 @@
 2013-09-08  Filip Pizlo  <fpizlo@apple.com>
 
+        run-jsc-stress-tests should be more paranoid about calling make
+        https://bugs.webkit.org/show_bug.cgi?id=121011
+
+        Reviewed by Oliver Hunt.
+        
+        If it returns with any kind of error then bail out.  Also print the command being used
+        in case we have to debug this, for example on the bots.  The extra verbosity isn't a
+        big deal.
+
+        * Scripts/run-jsc-stress-tests:
+
+2013-09-08  Filip Pizlo  <fpizlo@apple.com>
+
         Unreviewed, never pass Pathname to shellescape() as old versions of Ruby don't
         like that. Pass a string instead.
 
index 964a909..441c786 100755 (executable)
@@ -36,6 +36,16 @@ rescue => e
     $stderr.puts "Error: #{e.inspect}"
 end
 
+def mysys(*cmd)
+    begin
+        commandArray = cmd.each{|value| Shellwords.shellescape(value.to_s)}.join(' ')
+    rescue
+        commandArray = cmd.join(' ')
+    end
+    $stderr.puts ">> #{commandArray}"
+    raise "Command failed: #{$?.inspect}" unless system(*cmd)
+end
+
 numProcessors = `sysctl -n hw.availcpu`.to_i
 
 $jscPath = nil
@@ -319,7 +329,7 @@ if $parallel
     }
     
     Dir.chdir(parallelDir) {
-        system("make", "-j", numProcessors.to_s, "-s", "-f", "Makefile")
+        mysys("make", "-j", numProcessors.to_s, "-s", "-f", "Makefile")
     }
     
     Dir.foreach(parallelDir) {