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

Unified Diff: native_client_sdk/src/build_tools/test_projects.py

Issue 720233003: [NaCl SDK] Convert python scripts from optparse to argparse. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years 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
Index: native_client_sdk/src/build_tools/test_projects.py
diff --git a/native_client_sdk/src/build_tools/test_projects.py b/native_client_sdk/src/build_tools/test_projects.py
index 6d29cf8e28164bb8c26658f77e8f98c75fdfe644..f58cac111a264abdeec5572bc594bd08cdcb4e7e 100755
--- a/native_client_sdk/src/build_tools/test_projects.py
+++ b/native_client_sdk/src/build_tools/test_projects.py
@@ -3,7 +3,7 @@
# Use of this source code is governed by a BSD-style license that can be
# found in the LICENSE file.
-import optparse
+import argparse
import os
import subprocess
import sys
@@ -309,25 +309,26 @@ def GetProjectTree(include):
def main(args):
- parser = optparse.OptionParser()
- parser.add_option('-c', '--config',
+ parser = argparse.ArgumentParser(description=__doc__)
+ parser.add_argument('-c', '--config',
help='Choose configuration to run (Debug or Release). Runs both '
'by default', action='append')
- parser.add_option('-x', '--experimental',
+ parser.add_argument('-x', '--experimental',
help='Run experimental projects', action='store_true')
- parser.add_option('-t', '--toolchain',
+ parser.add_argument('-t', '--toolchain',
help='Run using toolchain. Can be passed more than once.',
action='append', default=[])
- parser.add_option('-d', '--dest',
+ parser.add_argument('-d', '--dest',
help='Select which destinations (project types) are valid.',
action='append')
- parser.add_option('-b', '--build',
+ parser.add_argument('-b', '--build',
help='Build each project before testing.', action='store_true')
- parser.add_option('--retry-times',
+ parser.add_argument('--retry-times',
help='Number of types to retry on failure (Default: %default)',
- type='int', default=1)
+ type=int, default=1)
+ parser.add_argument('projects', nargs='*')
- options, args = parser.parse_args(args)
+ options = parser.parse_args(args)
if not options.toolchain:
options.toolchain = ['newlib', 'glibc', 'pnacl', 'host']
@@ -348,9 +349,9 @@ def main(args):
if options.dest:
include['DEST'] = options.dest
print 'Filter by type: ' + str(options.dest)
- if args:
- include['NAME'] = args
- print 'Filter by name: ' + str(args)
+ if options.projects:
+ include['NAME'] = options.projects
+ print 'Filter by name: ' + str(options.projects)
if not options.config:
options.config = ALL_CONFIGS

Powered by Google App Engine
This is Rietveld 408576698