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

Unified Diff: src/scripts/autotest

Issue 1582017: Fix 'autotest' defaults -- i.e., fail when tests fail to build, (Closed)
Patch Set: Created 10 years, 8 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: src/scripts/autotest
diff --git a/src/scripts/autotest b/src/scripts/autotest
index 0d28dd632b7cc282b1b00aa90407f544d36493f9..0c8d71fe7ffed496c1d37122fa78e63e7b8630f3 100755
--- a/src/scripts/autotest
+++ b/src/scripts/autotest
@@ -58,14 +58,20 @@ parser.allow_interspersed_args = True
DEFAULT_BOARD = os.environ.get('DEFAULT_BOARD', '')
parser.add_option('--autox', dest='autox', action='store_true',
- help='Build autox along with autotest.')
+ default=True,
+ help='Build autox along with autotest [default].')
+parser.add_option('--noautox', dest='autox', action='store_false',
+ help='Don\'t build autox along with autotest.')
parser.add_option('--board', dest='board', action='store',
default=DEFAULT_BOARD,
help='The board for which you are building autotest.')
parser.add_option('--build', dest='build', action='store',
help='Only prebuild client tests, do not run tests.')
parser.add_option('--buildcheck', dest='buildcheck', action='store_true',
- help='Fail if tests fail to build.')
+ default=True,
+ help='Fail if tests fail to build [default].')
+parser.add_option('--nobuildcheck', dest='buildcheck', action='store_false',
+ help='Ignore test build failures.')
parser.add_option('--jobs', dest='jobs', action='store', type=int,
default=-1,
help='How many packages to build in parallel at maximum.')
@@ -212,4 +218,3 @@ def main():
if __name__ == '__main__':
main()
-
« 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