Scrub step names to avoid breaking buildbot features.

dart/utils/compiler/buildbot.py

Index: dart/utils/compiler/buildbot.py
diff --git a/dart/utils/compiler/buildbot.py b/dart/utils/compiler/buildbot.py
index ba42afae6aec3589d552b20ed6727ce19e32ef90..c9a74a6d29355e0ceec663a7d3febf3dfd7e1da1 100644
--- a/dart/utils/compiler/buildbot.py
+++ b/dart/utils/compiler/buildbot.py
@@ -100,9 +100,14 @@ def GetBuildInfo():
 def NeedsXterm(compiler, runtime):
   return runtime in ['ie', 'chrome', 'safari', 'opera', 'ff', 'drt']
 
+
+def TestStepName(name, flags):

[ricow1, 2012/09/09 17:55:57]

add short comment explaining that having "=" in the step name will break the
buildbots ability to correctly extract timing stats for the step

[ahe, 2012/09/09 18:06:02]

Done.

+  flags = [x for x in flags if not '=' in x]
+  return '%s tests %s' % (name, ' '.join(flags))
+
+
 def TestStep(name, mode, system, compiler, runtime, targets, flags):
-  print '@@@BUILD_STEP %s %s tests: %s %s@@@' % (name, compiler, runtime,
-      ' '.join(flags))
+  print '@@@BUILD_STEP %s@@@' % TestStepName(name, flags)
   sys.stdout.flush()
   if NeedsXterm(compiler, runtime) and system == 'linux':
     cmd = ['xvfb-run', '-a']