laforge has uploaded this change for review. ( https://gerrit.osmocom.org/c/pysim/+/28763 )

Change subject: pySim-trace: Fix --no-suppress-{select.status} command line arguments ......................................................................

pySim-trace: Fix --no-suppress-{select.status} command line arguments

The Tracer implemented those options and the argparser handled it, but we didn't ever connect the two.

Change-Id: I7d7d5fc475a8d09efdb63d3d6f1cc1de1996687b --- M pySim-trace.py 1 file changed, 5 insertions(+), 3 deletions(-)

git pull ssh://gerrit.osmocom.org:29418/pysim refs/changes/63/28763/1

diff --git a/pySim-trace.py b/pySim-trace.py index 8ee82f3..c480168 100755 --- a/pySim-trace.py +++ b/pySim-trace.py @@ -117,8 +117,10 @@ formatter_class=argparse.ArgumentDefaultsHelpFormatter)

global_group = option_parser.add_argument_group('General Options') -global_group.add_argument('--no-suppress-select', help="Don't suppress displaying SELECT APDUs") -global_group.add_argument('--no-suppress-status', help="Don't suppress displaying STATUS APDUs") +global_group.add_argument('--no-suppress-select', action='store_false', dest='suppress_select', + help="Don't suppress displaying SELECT APDUs") +global_group.add_argument('--no-suppress-status', action='store_false', dest='suppress_status', + help="Don't suppress displaying STATUS APDUs")

subparsers = option_parser.add_subparsers(help='APDU Source', dest='source', required=True)

@@ -153,7 +155,7 @@ elif opts.source == 'rspro-pyshark-live': s = PysharkRsproLive(opts.interface)

- tracer = Tracer(source=s) + tracer = Tracer(source=s, suppress_status=opts.suppress_status, suppress_select=opts.suppress_select) logger.info('Entering main loop...') tracer.main()