Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(641)

Unified Diff: dart/utils/compiler/buildbot.py

Issue 10905208: Add --step_name option to test.dart. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge
Patch Set: Created 8 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« dart/tools/test.dart ('K') | « dart/tools/testing/dart/test_runner.dart ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: dart/utils/compiler/buildbot.py
diff --git a/dart/utils/compiler/buildbot.py b/dart/utils/compiler/buildbot.py
index 6da39b1e23e24ecc63f3a246ece894744dfc2458..5c43f237f105b63e0381b5a00f40eb230a156d23 100644
--- a/dart/utils/compiler/buildbot.py
+++ b/dart/utils/compiler/buildbot.py
@@ -109,7 +109,7 @@ def TestStepName(name, flags):
def TestStep(name, mode, system, compiler, runtime, targets, flags):
- print '@@@BUILD_STEP %s@@@' % TestStepName(name, flags)
+ step_name = TestStepName(name, flags)
sys.stdout.flush()
if NeedsXterm(compiler, runtime) and system == 'linux':
cmd = ['xvfb-run', '-a']
@@ -120,6 +120,7 @@ def TestStep(name, mode, system, compiler, runtime, targets, flags):
ricow1 2012/09/11 08:14:38 Why don't we print the @@@BUILD_STEP $step_name@@@
cmd.extend([sys.executable,
os.path.join(os.curdir, 'tools', 'test.py'),
+ '--step_name=' + step_name,
'--mode=' + mode,
'--compiler=' + compiler,
'--runtime=' + runtime,
« dart/tools/test.dart ('K') | « dart/tools/testing/dart/test_runner.dart ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698