Adds saner behavior for git try -t foo to not fail in a inexplicable way. Enforces that each builde…

trychange.py

Index: trychange.py
diff --git a/trychange.py b/trychange.py
index 748dbfa6cd45f90c68ab69fcc255f238645fe3e3..9c449ae49524e44661fced57c01080de0abe2c16 100755
--- a/trychange.py
+++ b/trychange.py
@@ -805,6 +805,16 @@ def TryChange(argv,
             sys.stdout)
       except ImportError:
         pass
+      if options.testfilter:
+        bots = set()
+        for bot in options.bot:
+          assert ',' not in bot
+          if bot.endswith(':compile'):
+            # Strip it.

[Peter Mayo, 2012/03/10 00:30:13]

This comment is not clear - the code says skip the bot.  The comment implies
strip the ":compile"

The code seem right for most testfilters, though some might argue that
implicitly keeping a compile target on that bot would be closer to optimal
intent tracking.

[M-A Ruel, 2012/03/10 01:40:45]

For now I'll keep it this way but I'll probably revisit soon.

+            continue
+          bots.add(bot.split(':', 1)[0])
+        options.bot = list(bots)
+
       # If no bot is specified, either the default pool will be selected or the
       # try server will refuse the job. Either case we don't need to interfere.