Attachment 340033 Details for Bug 185492 – Patch

[patch] Patch

bug-185492-20180509144759.patch (text/plain), 2.21 KB, created by Stephan Szabo on 2018-05-09 14:48:00 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Stephan Szabo

Created: 2018-05-09 14:48:00 PDT

Size: 2.21 KB

patch

obsolete

>Subversion Revision: 231381
>diff --git a/Tools/ChangeLog b/Tools/ChangeLog
>index 77bb27188b7fc6dee70821728fcb2ced24607bd0..42db82cc088c286f0dd7759787b56b07ea0590f0 100644
>--- a/Tools/ChangeLog
>+++ b/Tools/ChangeLog
>@@ -1,3 +1,13 @@
>+2018-05-09  Stephan Szabo  <stephan.szabo@sony.com>
>+
>+        Support --verbose in run-javascriptcore-tests
>+        https://bugs.webkit.org/show_bug.cgi?id=185492
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        * Scripts/run-javascriptcore-tests:
>+        (runJSCStressTests):
>+
> 2018-05-04  Leo Balter  <leonardo.balter@gmail.com>
> 
>         Test262-Runner: Add base module to the local deps - unicore/PVA.pl
>diff --git a/Tools/Scripts/run-javascriptcore-tests b/Tools/Scripts/run-javascriptcore-tests
>index 75f16a475566a26664063776fa631d9d69c54cc7..e3419c56f44993bf9ffa48e62af3bb506566a1aa 100755
>--- a/Tools/Scripts/run-javascriptcore-tests
>+++ b/Tools/Scripts/run-javascriptcore-tests
>@@ -93,6 +93,7 @@ my $failFast = 1;
> my %jsonData = ();
> my $remoteConfigFile;
> my $jsonFileName;
>+my $verbose = 0;
> 
> if ($ENV{RUN_JAVASCRIPTCORE_TESTS_TESTMASM}) {
>     if ($ENV{RUN_JAVASCRIPTCORE_TESTS_TESTMASM} eq "true") {
>@@ -231,6 +232,7 @@ Usage: $programName [options] [options to pass to build system]
>                                 Each environment variable should be separated by a space.
>                                 e.g. \"foo=bar x=y\" (no quotes).
>   --gmalloc:                    Run tests with Guard Malloc enabled (if no path is given: $gmallocDefaultPath is used)
>+  --verbose:                    Print more things while running. (can be specified multiple times)
> 
> Environment Variables:
>   - set RUN_JAVASCRIPTCORE_TESTS_TESTMASM to "true" or "false" (no quotes) to determine if we run testmasm by default.
>@@ -277,6 +279,7 @@ GetOptions(
>     'help' => \$showHelp,
>     'env-vars=s' => \$envVars,
>     'gmalloc:s' => \$gmallocPath,
>+    'verbose+' => \$verbose
> );
> 
> 
>@@ -517,6 +520,10 @@ sub runJSCStressTests
>         push(@jscStressDriverCmd, $filter);
>     }
> 
>+    if ($verbose > 0) {
>+        push(@jscStressDriverCmd, ("--verbose") x $verbose)
>+    }
>+
>     unshift @jscStressDriverCmd, wrapperPrefixIfNeeded() if shouldUseJhbuild();
> 
>     # End option processing, the rest of the arguments are tests

Flags:

dbates: review+

Actions: View | Formatted Diff | Diff

Attachments on bug 185492: 340033 | 340098