Attachment 339819 Details for Bug 185392 – Patch

[patch] Patch

bug-185392-20180508103109.patch (text/plain), 4.81 KB, created by valerie on 2018-05-08 07:31:10 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: valerie

Created: 2018-05-08 07:31:10 PDT

Size: 4.81 KB

patch

obsolete

>Subversion Revision: 231482
>diff --git a/Tools/ChangeLog b/Tools/ChangeLog
>index 2f11cca080a23888aa0c0aeb6ff5c67e5e42219b..f48e3a2b848ab6e39a09160092052cfa8e823336 100644
>--- a/Tools/ChangeLog
>+++ b/Tools/ChangeLog
>@@ -1,3 +1,13 @@
>+2018-05-07  Valerie R Young  <valerie@bocoup.com>
>+
>+        test262/Runner.pm: create results dir for results of test262 run
>+        https://bugs.webkit.org/show_bug.cgi?id=185392
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        * Scripts/test262/Runner.pm:
>+        (processCLI):
>+
> 2018-05-07  Chris Dumez  <cdumez@apple.com>
> 
>         Unreviewed, fix issue with running Speedometer PerfTest after r231450.
>diff --git a/Tools/Scripts/test262/Runner.pm b/Tools/Scripts/test262/Runner.pm
>index 2d0a3275033f3ad82148136636e6763941203a7c..1b9eb8b5c4da25787268d460d63f70795961f171 100755
>--- a/Tools/Scripts/test262/Runner.pm
>+++ b/Tools/Scripts/test262/Runner.pm
>@@ -36,6 +36,7 @@ use File::Find;
> use File::Temp qw(tempfile tempdir);
> use File::Spec::Functions qw(abs2rel);
> use File::Basename qw(dirname);
>+use File::Path qw(mkpath);
> use Cwd qw(abs_path);
> use FindBin;
> use Env qw(DYLD_FRAMEWORK_PATH);
>@@ -84,9 +85,15 @@ my $runningAllTests;
> 
> my $expectationsFile = abs_path("$Bin/expectations.yaml");
> my $configFile = abs_path("$Bin/config.yaml");
>-my $resultsFile = abs_path("$Bin/results.yaml");
>-my $summaryTxtFile = abs_path("$Bin/results-summary.txt");
>-my $summaryFile = abs_path("$Bin/results-summary.yaml");
>+
>+my $resultsDir = `pwd`;
>+chomp $resultsDir;
>+$resultsDir = $resultsDir . "/test262-results";
>+mkpath($resultsDir);
>+
>+my $resultsFile = abs_path("$resultsDir/results.yaml");
>+my $summaryTxtFile = abs_path("$resultsDir/summary.txt");
>+my $summaryFile = abs_path("$resultsDir/summary.yaml");
> 
> my @results;
> my @files;
>@@ -106,6 +113,7 @@ sub processCLI {
>     my $ignoreExpectations;
>     my @features;
>     my $stats;
>+    my $specifiedResultsFile;
> 
>     # If adding a new commandline argument, you must update the POD
>     # documentation at the end of the file.
>@@ -125,6 +133,7 @@ sub processCLI {
>         'failing-files' => \$failingOnly,
>         'l|latest-import' => \$latestImport,
>         'stats' => \$stats,
>+        'r|results=s' => \$specifiedResultsFile,
>     );
> 
>     if ($help) {
>@@ -136,7 +145,22 @@ sub processCLI {
>         }
>     }
> 
>+    if ($specifiedResultsFile) {
>+        if (!$stats) {
>+            print "Waring: supplied results file not used for this command.\n";
>+        }
>+        elsif (-e $specifiedResultsFile) {
>+            $resultsFile = $specifiedResultsFile;
>+        }
>+        else {
>+            die "Error: results file $specifiedResultsFile does not exist.";
>+        }
>+    }
>+
>     if ($stats) {
>+        if (! -e $resultsFile) {
>+            die "Error: cannot find results file, please specify with --results.";
>+        }
>         summarizeResults();
>         exit;
>     }
>@@ -183,7 +207,7 @@ sub processCLI {
>         # If expectations file doesn't exist yet, just run tests, UNLESS
>         # --failures-only option supplied.
>         if ( $failingOnly && ! -e $expectationsFile ) {
>-            print "Error: Cannot run failing tests if test262-expectation.yaml file does not exist.\n";
>+            print "Error: Cannot run failing tests if expectation.yaml file does not exist.\n";
>             die;
>         } elsif (-e $expectationsFile) {
>             $expect = LoadFile($expectationsFile) or die $!;
>@@ -880,7 +904,11 @@ Runs the test files listed in the last import (./JSTests/test262/latest-changes-
> 
> =item B<--stats>
> 
>-Calculate conformance statistics from JSTests/test262-results.yaml file. Saves results in JSTests/test262/results-summary.txt and JSTests/test262/results-summary.yaml.
>+Calculate conformance statistics from results/results.yaml file or a supplied results file (--results). Saves results in results/summary.txt and results/summary.yaml.
>+
>+=item B<--results, -r>
>+
>+Specifies a results file the --stats option.
> 
> =back
> 
>diff --git a/.gitignore b/.gitignore
>index 9e16334c90bdeb05e68a1db8554b0d54e70d17f6..e934031387bebcbfa0cb138f26d276b87c5681e0 100644
>--- a/.gitignore
>+++ b/.gitignore
>@@ -48,8 +48,3 @@ __cmake_systeminformation/
> 
> # Ignore YouCompleteMe symlinks
> .ycm_extra_conf.py
>-
>-# Ignore generated Test262 results files
>-Tools/Scripts/test262/results.yaml
>-Tools/Scripts/test262/results-summary.yaml
>-Tools/Scripts/test262/results-summary.txt
>diff --git a/ChangeLog b/ChangeLog
>index a146e60ae572e10a7fd85667c90aa9b490e1f27c..db49933dc461b2a9baa372c0f4a4ebf20bcaddad 100644
>--- a/ChangeLog
>+++ b/ChangeLog
>@@ -1,3 +1,12 @@
>+2018-05-07  Valerie R Young  <valerie@bocoup.com>
>+
>+        test262/Runner.pm: create results dir for results of test262 run
>+        https://bugs.webkit.org/show_bug.cgi?id=185392
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        * .gitignore:
>+
> 2018-05-07  Yusuke Suzuki  <utatane.tea@gmail.com>
> 
>         [JSCOnly] Enable Intl.PluralRules

Actions: View | Formatted Diff | Diff

Attachments on bug 185392: 339749 | 339819 | 339823 | 339858 | 339862 | 339864 | 339865 | 339870 | 339872