commit 72c5b2d7966707c97ceedce04cad6dff065fa3c0
author Harald Welte <laforge@osmocom.org> Sun Jul 24 10:21:41 2022 +0200
committer Harald Welte <laforge@osmocom.org> Sun Jul 24 10:23:50 2022 +0200
tree 022caf10e99c382ec3bd809b8f8f1fc7c2cd43d6
parent c61fbf4daa9e1d16e7e50ee199fa2f269ca45d02

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

pySim-trace.py

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()