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

Unified Diff: trychange.py

Issue 9428047: Improve trychange.py documentation. No logic change (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
diff --git a/trychange.py b/trychange.py
index ead63c338d32956a5a90613179a4deed92de8e8f..4d832aca5505312aad89f4f907cef0b9c70fd83f 100755
--- a/trychange.py
+++ b/trychange.py
@@ -535,10 +535,13 @@ def TryChange(argv,
parser.add_option_group(group)
group = optparse.OptionGroup(parser, "Try job options")
- group.add_option("-b", "--bot", action="append",
- 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", "--bot", action="append",
+ help=("Specify builder and comma separated tests. Can be used multiple "
+ "times. ex: '--bot win_rel:ui_tests,webkit_unit_tests --bot "
+ "win_layout'; see the try server waterfall for the builders name "
+ "and the tests available. Can also be used with gtest_filter, e.g. "
+ "'--bot win_rel:base_unittests:ValuesTest.*Value'"))
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. Like --dry_run"
@@ -558,13 +561,11 @@ def TryChange(argv,
help="Override which project to use. Projects are defined "
"server-side to define what default bot set to use")
- group.add_option("-t", "--testfilter", action="append", default=[],
- help="Add a gtest_filter to a test. Use multiple times to "
- "specify filters for different tests. (i.e. "
- "--testfilter base_unittests:ThreadTest.* "
- "--testfilter ui_tests) If you specify any testfilters "
- "the test results will not be reported in rietveld and "
- "only tests with filters will run.")
+ group.add_option(
+ "-t", "--testfilter", action="append", default=[],
+ help=("Apply a testfilter to all the selected builders. Unless the "
+ "builders configurations are similar, it is preferable to use "
Peter Mayo 2012/02/22 20:45:09 suggest s/similar/the same/ s/it is preferable to
+ "multiple --bot <builder>:<test> arguments."))
parser.add_option_group(group)
« 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