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

Unified Diff: trychange.py

Issue 9348068: --print_bots arg to 'gcl try' for debugging PRESUBMIT.py changes. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/depot_tools/
Patch Set: Created 8 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: trychange.py
===================================================================
--- trychange.py (revision 120947)
+++ trychange.py (working copy)
@@ -539,6 +539,9 @@
help="Only use specifics build slaves, ex: "
"'--bot win,layout_mac'; see the try "
"server waterfall for the slave's name")
+ group.add_option("-B", "--print_bots", action="store_true",
+ help="Print bots we would use (e.g. from PRESUBMIT.py)"
+ " and exit. Do not send patch.")
M-A Ruel 2012/02/08 21:22:55 It's a bit confusing with the fact that --dry_run
group.add_option("-r", "--revision",
help="Revision to use for the try job; default: the "
"revision will be determined by the try server; see "
@@ -788,6 +791,10 @@
# 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.
+ if options.print_bots:
+ print 'Bots which would be used: ' + str(options.bot)
M-A Ruel 2012/02/08 21:22:55 I'd prefer: print 'Bots which would be used:' for
+ return 0
+
# Send the patch.
if options.send_patch:
# If forced.
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698