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

Unified Diff: scripts/master/try_job_svn.py

Issue 39167: Let the server accept multiple trybots. (Closed) Base URL: svn://chrome-svn/chrome/trunk/tools/buildbot/
Patch Set: Created 11 years, 10 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
« no previous file with comments | « scripts/master/try_job_http.py ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: scripts/master/try_job_svn.py
===================================================================
--- scripts/master/try_job_svn.py (revision 10943)
+++ scripts/master/try_job_svn.py (working copy)
@@ -29,7 +29,6 @@
self.watcher.setServiceParent(self)
def parseJob(self, comment, diff):
- builderNames = []
options = dict(item.split('=') for item in comment.splitlines())
job_name = options.get('name', 'Unnamed')
user = options.get('user', 'John Doe')
@@ -45,7 +44,7 @@
buildsetID = options.get('reason', "%s: %s" % (user, job_name))
builderNames = []
if 'bot' in options:
- builderNames = [ options['bot'] ]
+ builderNames = options['bot'].split(',')
# TODO(maruel): Don't select the builders right now if not specified.
builderNames = self.pools.Select(builderNames)
tests = options.get('tests', None)
« no previous file with comments | « scripts/master/try_job_http.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698