Reviewers: Jakob,

Description:
Fix test harness to correctly split extra flags.

[email protected]


Please review this at https://chromiumcodereview.appspot.com/12094032/

SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge

Affected files:
  M tools/run-tests.py
  M tools/testrunner/local/execution.py


Index: tools/run-tests.py
diff --git a/tools/run-tests.py b/tools/run-tests.py
index c2c43a9c8c0f6b99a4f30b45b144470dbb32fd9b..292f6f6c737b9fd16c5a7ac486310c516c06ed29 100755
--- a/tools/run-tests.py
+++ b/tools/run-tests.py
@@ -178,6 +178,7 @@ def ProcessOptions(options):
           "running tests locally.")
     options.no_network = True
   options.command_prefix = shlex.split(options.command_prefix)
+  options.extra_flags = shlex.split(options.extra_flags)
   if options.j == 0:
     options.j = multiprocessing.cpu_count()
   if options.no_stress:
Index: tools/testrunner/local/execution.py
diff --git a/tools/testrunner/local/execution.py b/tools/testrunner/local/execution.py index 9ca3991a04218d4a58d07bd6233a3b57dba983ff..0f52616d97fcc574caeeadf07c5a111684e324fc 100644
--- a/tools/testrunner/local/execution.py
+++ b/tools/testrunner/local/execution.py
@@ -171,7 +171,7 @@ class Runner(object):
            [os.path.abspath(os.path.join(self.context.shell_dir, shell))] +
            d8testflag +
            test.suite.GetFlagsForTestCase(test, self.context) +
-           [self.context.extra_flags])
+           self.context.extra_flags)
     return cmd




--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to