run-jsc-benchmarks' echo should have -e option
authorutatane.tea@gmail.com <utatane.tea@gmail.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 13 Jan 2017 19:00:29 +0000 (19:00 +0000)
committerutatane.tea@gmail.com <utatane.tea@gmail.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 13 Jan 2017 19:00:29 +0000 (19:00 +0000)
https://bugs.webkit.org/show_bug.cgi?id=167009

Reviewed by Sam Weinig.

echo requires `-e` option to use escape sequence.
Without it, in some environment (in my Fedora 25), the escape sequence is not
recognized and just dumped as `\c` etc.

* Scripts/run-jsc-benchmarks:

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

Tools/ChangeLog
Tools/Scripts/run-jsc-benchmarks

index 65afbbe..31f7914 100644 (file)
@@ -1,3 +1,16 @@
+2017-01-13  Yusuke Suzuki  <utatane.tea@gmail.com>
+
+        run-jsc-benchmarks' echo should have -e option
+        https://bugs.webkit.org/show_bug.cgi?id=167009
+
+        Reviewed by Sam Weinig.
+
+        echo requires `-e` option to use escape sequence.
+        Without it, in some environment (in my Fedora 25), the escape sequence is not
+        recognized and just dumped as `\c` etc.
+
+        * Scripts/run-jsc-benchmarks:
+
 2017-01-12  Chris Dumez  <cdumez@apple.com>
 
         Add KEYBOARD_KEY_ATTRIBUTE / KEYBOARD_CODE_ATTRIBUTE to FeatureDefines.xcconfig
index 6c5f2bf..83f92b3 100755 (executable)
@@ -3283,10 +3283,10 @@ begin
   if $prepare
     File.open("#{BENCH_DATA_PATH}/runscript", "w") {
       | file |
-      file.puts "echo \"HOSTNAME:\\c\""
+      file.puts "echo -e \"HOSTNAME:\\c\""
       file.puts "hostname"
       file.puts "echo"
-      file.puts "echo \"HARDWARE:\\c\""
+      file.puts "echo -e \"HARDWARE:\\c\""
       file.puts "/usr/sbin/sysctl hw.model"
       file.puts "echo"
       file.puts "set -e"
@@ -3296,14 +3296,14 @@ begin
         if $verbosity == 0 and not $silent
           text1 = lpad(idx.to_s,$runPlans.size.to_s.size)+"/"+$runPlans.size.to_s
           text2 = plan.to_s
-          file.puts("echo " + Shellwords.shellescape("\r#{text1} #{rpad(text2,$planpad)}") + "\"\\c\" 1>&2")
-          file.puts("echo " + Shellwords.shellescape("\r#{text1} #{text2}") + "\"\\c\" 1>&2")
+          file.puts("echo -e " + Shellwords.shellescape("\r#{text1} #{rpad(text2,$planpad)}") + "\"\\c\" 1>&2")
+          file.puts("echo -e " + Shellwords.shellescape("\r#{text1} #{text2}") + "\"\\c\" 1>&2")
         end
         plan.emitRunCode
       }
       if $verbosity == 0 and not $silent
-        file.puts("echo " + Shellwords.shellescape("\r#{$runPlans.size}/#{$runPlans.size} #{' '*($suitepad+1+$benchpad+1+$vmpad)}") + "\"\\c\" 1>&2")
-        file.puts("echo " + Shellwords.shellescape("\r#{$runPlans.size}/#{$runPlans.size}") + " 1>&2")
+        file.puts("echo -e " + Shellwords.shellescape("\r#{$runPlans.size}/#{$runPlans.size} #{' '*($suitepad+1+$benchpad+1+$vmpad)}") + "\"\\c\" 1>&2")
+        file.puts("echo -e " + Shellwords.shellescape("\r#{$runPlans.size}/#{$runPlans.size}") + " 1>&2")
       end
     }
   end