Attention is currently required from: laforge.
dexter has posted comments on this change by dexter. ( https://gerrit.osmocom.org/c/pysim/+/37895?usp=email )
Change subject: pySim-shell, cosmetic: define positional arguments last ......................................................................
Patch Set 2:
(1 comment)
File pySim-shell.py:
https://gerrit.osmocom.org/c/pysim/+/37895/comment/9e777c8f_33725b6e?usp=ema... : PS1, Line 238: apdu_cmd_parser = argparse.ArgumentParser()
is this really a cosmetic change? I would expect that before this change, the --opts would have to […]
To my experience this will not change the command syntax. To my experience it does not matter if positional arguments come first or last. I have tested it now again:
apdu 00a40004023f00 --expect-sw 9000 apdu --expect-sw 9000 00a40004023f00
both commands work the same. I also tried it out manually with some python code. The result is the same.
-
dexter