This is merely a historical archive of years 2008-2021, before the migration to mailman3.
A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.
laforge gerrit-no-reply at lists.osmocom.orglaforge has submitted this change. ( https://gerrit.osmocom.org/c/pysim/+/23718 )
Change subject: pySim-shell: Migrate from optparse to argparse for the main()
......................................................................
pySim-shell: Migrate from optparse to argparse for the main()
We're using argparse internally for all shell commands, and can
use that to auto-generate command reference in the manual.
Let's switch to argparse for the main program, too - and generate
the related reference in the manual.
Change-Id: I77c946dbeb9f746fe3d8051173e59462dc2fb5e2
---
M docs/shell.rst
M pySim-shell.py
2 files changed, 37 insertions(+), 54 deletions(-)
Approvals:
Jenkins Builder: Verified
fixeria: Looks good to me, approved
diff --git a/docs/shell.rst b/docs/shell.rst
index 30938fe..4cdf9e0 100644
--- a/docs/shell.rst
+++ b/docs/shell.rst
@@ -23,6 +23,20 @@
* tab completion of commands and SELECT-able files/directories
* interactive help for all commands
+Running pySim-shell
+-------------------
+
+pySim-shell has a variety of command line arguments to control
+
+* which transport to use (how to use a reader to talk to the SIM card)
+* whether to automatically verify an ADM pin (and in which format)
+* whether to execute a start-up script
+
+.. argparse::
+ :module: pySim-shell
+ :func: option_parser
+
+
cmd2 basics
-----------
diff --git a/pySim-shell.py b/pySim-shell.py
index 5298cb4..6bc9a95 100755
--- a/pySim-shell.py
+++ b/pySim-shell.py
@@ -28,7 +28,6 @@
import os
import sys
-from optparse import OptionParser
from pathlib import Path
from pySim.ts_51_011 import EF, DF, EF_SST_map, EF_AD_mode_map
@@ -430,64 +429,34 @@
(data, sw) = self._cmd.card._scc.manage_channel(mode='close', lchan_nr=opts.chan_nr)
-def parse_options():
-
- parser = OptionParser(usage="usage: %prog [options]")
-
- parser.add_option("-d", "--device", dest="device", metavar="DEV",
- help="Serial Device for SIM access [default: %default]",
- default="/dev/ttyUSB0",
- )
- parser.add_option("-b", "--baud", dest="baudrate", type="int", metavar="BAUD",
- help="Baudrate used for SIM access [default: %default]",
- default=9600,
- )
- parser.add_option("-p", "--pcsc-device", dest="pcsc_dev", type='int', metavar="PCSC",
- help="Which PC/SC reader number for SIM access",
- default=None,
- )
- parser.add_option("--modem-device", dest="modem_dev", metavar="DEV",
- help="Serial port of modem for Generic SIM Access (3GPP TS 27.007)",
- default=None,
- )
- parser.add_option("--modem-baud", dest="modem_baud", type="int", metavar="BAUD",
- help="Baudrate used for modem's port [default: %default]",
- default=115200,
- )
- parser.add_option("--osmocon", dest="osmocon_sock", metavar="PATH",
- help="Socket path for Calypso (e.g. Motorola C1XX) based reader (via OsmocomBB)",
- default=None,
- )
- parser.add_option("--script", dest="script", metavar="PATH",
- help="script with shell commands to be executed automatically",
- default=None,
- )
-
- parser.add_option("--csv", dest="csv", metavar="FILE",
- help="Read card data from CSV file",
- default=None,
- )
-
- parser.add_option("-a", "--pin-adm", dest="pin_adm",
- help="ADM PIN used for provisioning (overwrites default)",
- )
- parser.add_option("-A", "--pin-adm-hex", dest="pin_adm_hex",
- help="ADM PIN used for provisioning, as hex string (16 characters long",
- )
-
- (options, args) = parser.parse_args()
-
- if args:
- parser.error("Extraneous arguments")
-
- return options
-
+option_parser = argparse.ArgumentParser(prog='pySim-shell', description='interactive SIM card shell',
+ formatter_class=argparse.ArgumentDefaultsHelpFormatter)
+option_parser.add_argument('-d', '--device', metavar='DEV', default='/dev/ttyUSB0',
+ help='Serial Device for SIM access')
+option_parser.add_argument('-b', '--baud', dest='baudrate', type=int, metavar='BAUD', default=9600,
+ help='Baud rate used for SIM access')
+option_parser.add_argument('-p', '--pcsc-device', type=int, dest='pcsc_dev', metavar='PCSC', default=None,
+ help='PC/SC reader number to use for SIM access')
+option_parser.add_argument('--modem-device', dest='modem_dev', metavar='DEV', default=None,
+ help='Serial port of modem for Generic SIM Access (3GPP TS 27.007)')
+option_parser.add_argument('--modem-baud', type=int, metavar='BAUD', default=115200,
+ help='Baud rate used for modem port')
+option_parser.add_argument('--osmocon', dest='osmocon_sock', metavar='PATH', default=None,
+ help='Socket path for Calypso (e.g. Motorola C1XX) based reader (via OsmocomBB)')
+option_parser.add_argument('--script', metavar='PATH', default=None,
+ help='script with pySim-shell commands to be executed automatically at start-up')
+option_parser.add_argument('--csv', metavar='FILE', default=None,
+ help='Read card data from CSV file')
+option_parser.add_argument('-a', '--pin-adm', metavar='PIN_ADM1', dest='pin_adm', default=None,
+ help='ADM PIN used for provisioning (overwrites default)')
+option_parser.add_argument('-A', '--pin-adm-hex', metavar='PIN_ADM1_HEX', dest='pin_adm_hex', default=None,
+ help='ADM PIN used for provisioning, as hex string (16 characters long)')
if __name__ == '__main__':
# Parse options
- opts = parse_options()
+ opts = option_parser.parse_args()
# Init card reader driver
sl = init_reader(opts)
--
To view, visit https://gerrit.osmocom.org/c/pysim/+/23718
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: I77c946dbeb9f746fe3d8051173e59462dc2fb5e2
Gerrit-Change-Number: 23718
Gerrit-PatchSet: 1
Gerrit-Owner: laforge <laforge at osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <pmaier at sysmocom.de>
Gerrit-Reviewer: fixeria <vyanitskiy at sysmocom.de>
Gerrit-Reviewer: laforge <laforge at osmocom.org>
Gerrit-Reviewer: tnt <tnt at 246tNt.com>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20210412/9d8cf0e6/attachment.htm>