Attention is currently required from: dexter.
laforge 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 1: Code-Review-1
(1 comment)
File pySim-shell.py:
https://gerrit.osmocom.org/c/pysim/+/37895/comment/8b29a5ca_b1d7fa21?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 come after the APDU, while now they would have to come before the APDU? If this is the case, this would break tons of scripts I have here, e.g. for card personalization.
Please verify that this change still accepts the existing/old argument ordering.