run-jsc-stress-tests should be smart enough to call JSRegress "regress/script-tests...
authorfpizlo@apple.com <fpizlo@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 8 Sep 2013 17:31:34 +0000 (17:31 +0000)
committerfpizlo@apple.com <fpizlo@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 8 Sep 2013 17:31:34 +0000 (17:31 +0000)
https://bugs.webkit.org/show_bug.cgi?id=120994

Reviewed by Geoffrey Garen.

run-jsc-stress-tests accepts a list of directories containing tests.  It needs to
come up with names for all of them.  Currently it uses the basename of the test
directory.  But that means that JSRegress gets called "script-tests", which is
unfortunate since any directory in LayoutTests containing .js files will be
called "script-tests".  So, this changes run-jsc-stress-tests to use more than
just the basename if the basename has the word "tests" in it.  This causes
JSRegress to be called "regress/script-tests".

* Scripts/run-jsc-stress-tests:

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

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

index b38b9fd..fe01e89 100644 (file)
@@ -1,3 +1,20 @@
+2013-09-08  Filip Pizlo  <fpizlo@apple.com>
+
+        run-jsc-stress-tests should be smart enough to call JSRegress "regress/script-tests" rather than "script-tests"
+        https://bugs.webkit.org/show_bug.cgi?id=120994
+
+        Reviewed by Geoffrey Garen.
+        
+        run-jsc-stress-tests accepts a list of directories containing tests.  It needs to
+        come up with names for all of them.  Currently it uses the basename of the test
+        directory.  But that means that JSRegress gets called "script-tests", which is
+        unfortunate since any directory in LayoutTests containing .js files will be
+        called "script-tests".  So, this changes run-jsc-stress-tests to use more than
+        just the basename if the basename has the word "tests" in it.  This causes
+        JSRegress to be called "regress/script-tests".
+
+        * Scripts/run-jsc-stress-tests:
+
 2013-09-08  Sergio Correia  <sergio.correia@openbossa.org>
 
         [EFL][JHBUILD] Fix evas build with giflib5
index 3d58ed8..3fdd709 100755 (executable)
@@ -137,7 +137,7 @@ def defaultRun
 end
 
 def skip
-    puts "Skipping #{$benchmark}"
+    puts "Skipping #{$collectionName}/#{$benchmark}"
 end
 
 collectionNames = {}
@@ -152,17 +152,29 @@ $outputDir = $outputDir.realpath
 
 ARGV.each {
     | collection |
-    $collection = collection
-    $collectionName = File.basename(collection)
+    $collection = Pathname.new(collection)
+    outerDir = $collection.dirname
+    name = $collection.basename
+    lastName = name
+    while lastName.to_s =~ /test/
+        lastName = outerDir.basename
+        name = lastName + name
+        outerDir = outerDir.dirname
+    end
+    $collectionName = name.to_s
     toAdd = 1
     while collectionNames[$collectionName]
-        $collectionName = File.basename(collection) + "-#{toAdd}"
+        $collectionName = File.basename(name.to_s) + "-#{toAdd}"
         toAdd += 1
     end
     collectionNames[$collectionName] = true
-    dir = $outputDir + $collectionName
-    Dir.mkdir(dir) unless dir.directory?
-    Dir.chdir(collection) {
+    dir = $outputDir
+    name.each_filename {
+        | filename |
+        dir = dir + filename
+        Dir.mkdir(dir) unless dir.directory?
+    }
+    Dir.chdir($collection) {
         Dir.foreach('.') {
             | benchmark |
             next unless benchmark =~ /\.js$/