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

Unified Diff: native_client_sdk/src/tools/decode_dump.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, 1 month 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/tools/decode_dump.py
diff --git a/native_client_sdk/src/tools/decode_dump.py b/native_client_sdk/src/tools/decode_dump.py
index 74d07a03fb6699d059d7cd36c4d8a114ee23eeb9..1adc0a5854014c76513cd31047080f63748dbdd7 100755
--- a/native_client_sdk/src/tools/decode_dump.py
+++ b/native_client_sdk/src/tools/decode_dump.py
@@ -9,7 +9,7 @@ Currently this produces a simple stack trace.
"""
import json
-import optparse
+import argparse
binji 2014/11/13 23:57:03 sort
Sam Clegg 2014/11/30 17:55:12 Done.
import os
import posixpath
import subprocess
@@ -177,18 +177,18 @@ class CoreDecoder(object):
def Main(args):
- parser = optparse.OptionParser(
+ parser = argparse.ArgumentParser(
usage='USAGE: %prog [options] <core.json>')
binji 2014/11/13 23:57:03 remove usage
Sam Clegg 2014/11/30 17:55:12 Done.
- parser.add_option('-m', '--main-nexe', dest='main_nexe',
+ parser.add_argument('-m', '--main-nexe', dest='main_nexe',
help='nexe to resolve NaClMain references from')
- parser.add_option('-n', '--nmf', dest='nmf_filename', default='-',
+ parser.add_argument('-n', '--nmf', dest='nmf_filename', default='-',
help='nmf to resolve references from')
- parser.add_option('-a', '--addr2line', dest='addr2line',
+ parser.add_argument('-a', '--addr2line', dest='addr2line',
help='path to appropriate addr2line')
- parser.add_option('-L', '--library-path', dest='library_paths',
+ parser.add_argument('-L', '--library-path', dest='library_paths',
action='append', default=[],
help='path to search for shared libraries')
- parser.add_option('-p', '--platform', dest='platform',
+ parser.add_argument('-p', '--platform', dest='platform',
help='platform in a style match nmf files')
options, args = parser.parse_args(args)
binji 2014/11/13 23:57:03 options = ...
Sam Clegg 2014/11/30 17:55:12 Done.
if len(args) != 1:
binji 2014/11/13 23:57:03 add positional arg
Sam Clegg 2014/11/30 17:55:12 Done.

Powered by Google App Engine
This is Rietveld 408576698