Update buildbot.py to support jsshell builders.

utils/compiler/buildbot.py

Index: utils/compiler/buildbot.py
===================================================================
--- utils/compiler/buildbot.py	(revision 13274)
+++ utils/compiler/buildbot.py	(working copy)
@@ -25,7 +25,7 @@
     os.path.dirname(os.path.dirname(os.path.abspath(__file__))))
 
 DART2JS_BUILDER = (
-    r'dart2js-(linux|mac|windows)-(debug|release)(-(checked|host-checked))?(-(host-checked))?-?(\d*)-?(\d*)')
+    r'dart2js-(linux|mac|windows)(-(jsshell))?-(debug|release)(-(checked|host-checked))?(-(host-checked))?-?(\d*)-?(\d*)')
 WEB_BUILDER = (
     r'dart2js-(ie|ff|safari|chrome|opera)-(win7|win8|mac|linux)(-(all|html))?')
 
@@ -113,22 +113,24 @@
       test_set = web_pattern.group(4)
     elif dart2js_pattern:
       compiler = 'dart2js'
+      system = dart2js_pattern.group(1)
       runtime = 'd8'
-      system = dart2js_pattern.group(1)
-      mode = dart2js_pattern.group(2)
+      if dart2js_pattern.group(3) == 'jsshell':

[ricow1, 2012/10/05 12:29:36]

I can change this, but that requires us to change the names of the existing
builders (i.e., explicitly requiring the runtime from the name, so
dart2js-linux-release-1-5 would be dart2js-linux-d8-release-1-5)
Let me know what you think
(I am not to happy about this naming pattern matching, it is getting more or
less unreadable for people who don't know this, I suggest that we refactor this
when we have a more stable set of builders, that way we have the full picture)

+        runtime = 'jsshell'
+      mode = dart2js_pattern.group(4)
       # The valid naming parts for checked and host-checked are:
       # Empty: checked=False, host_checked=False
       # -checked: checked=True, host_checked=False
       # -host-checked: checked=False, host_checked=True
       # -checked-host-checked: checked=True, host_checked=True
-      if dart2js_pattern.group(4) == 'checked':
+      if dart2js_pattern.group(6) == 'checked':
         checked = True
-      if dart2js_pattern.group(4) == 'host-checked':
-        host_checked = True
       if dart2js_pattern.group(6) == 'host-checked':
         host_checked = True
-      shard_index = dart2js_pattern.group(7)
-      total_shards = dart2js_pattern.group(8)
+      if dart2js_pattern.group(8) == 'host-checked':
+        host_checked = True
+      shard_index = dart2js_pattern.group(9)
+      total_shards = dart2js_pattern.group(10)
 
   if system == 'windows':
     system = 'win7'
@@ -207,7 +209,7 @@
 def TestCompiler(runtime, mode, system, flags, is_buildbot, test_set):
   """ test the compiler.
    Args:
-     - runtime: either 'd8', or one of the browsers, see GetBuildInfo
+     - runtime: either 'd8', 'jsshell', or one of the browsers, see GetBuildInfo
      - mode: either 'debug' or 'release'
      - system: either 'linux', 'mac', or 'win7'
      - flags: extra flags to pass to test.dart