Title: [157036] trunk/Tools
Revision
157036
Author
fpi...@apple.com
Date
2013-10-07 07:44:31 -0700 (Mon, 07 Oct 2013)

Log Message

run-jsc-stress-tests shouldn't print its goofy progress meter when running on the bots
https://bugs.webkit.org/show_bug.cgi?id=122437

Reviewed by Antoine Quint.

* Scripts/run-jsc-stress-tests:

Modified Paths

Diff

Modified: trunk/Tools/ChangeLog (157035 => 157036)


--- trunk/Tools/ChangeLog	2013-10-07 14:36:49 UTC (rev 157035)
+++ trunk/Tools/ChangeLog	2013-10-07 14:44:31 UTC (rev 157036)
@@ -1,5 +1,14 @@
 2013-10-06  Filip Pizlo  <fpi...@apple.com>
 
+        run-jsc-stress-tests shouldn't print its goofy progress meter when running on the bots
+        https://bugs.webkit.org/show_bug.cgi?id=122437
+
+        Reviewed by Antoine Quint.
+
+        * Scripts/run-jsc-stress-tests:
+
+2013-10-06  Filip Pizlo  <fpi...@apple.com>
+
         Unreviewed, fix build on old Rubies.
 
         * Scripts/run-jsc-stress-tests:

Modified: trunk/Tools/Scripts/run-jsc-stress-tests (157035 => 157036)


--- trunk/Tools/Scripts/run-jsc-stress-tests	2013-10-07 14:36:49 UTC (rev 157035)
+++ trunk/Tools/Scripts/run-jsc-stress-tests	2013-10-07 14:44:31 UTC (rev 157036)
@@ -123,6 +123,8 @@
     end
 }
 
+$progressMeter = ($verbosity == 0 and $stdin.tty?)
+
 unless $jscPath
     $stderr.puts "Error: must specify -j <path>."
     exit 1
@@ -303,7 +305,7 @@
     end
     
     def successCommand
-        if $verbosity == 0 or $verbosity >= 2
+        if $progressMeter or $verbosity >= 2
             "rm -f #{failFile} ; echo PASS: #{Shellwords.shellescape(@name)}"
         else
             "rm -f #{failFile}"
@@ -719,7 +721,7 @@
 }
 
 Dir.chdir(parallelDir) {
-    if $verbosity >= 1
+    unless $progressMeter
         mysys("make", "-j", numProcessors.to_s, "-s", "-f", "Makefile")
     else
         cmd = "make -j #{numProcessors} -s -f Makefile"
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to