Attention is currently required from: fixeria, laforge.

dexter has posted comments on this change by fixeria. ( https://gerrit.osmocom.org/c/pysim/+/42310?usp=email )

Change subject: global_platform: install_cap_parser: argument groups cannot be nested ......................................................................

Patch Set 2: Verified+1

(2 comments)

Patchset:

PS2: I have tested this patch, everything still works fine.

File pySim/global_platform/__init__.py:

https://gerrit.osmocom.org/c/pysim/+/42310/comment/529fa6d8_68998018?usp=ema... : PS2, Line 905: raise ValueError("Either --install-parameters alone, or all --install-parameters-* must be specified") I would write "--install-parameters cannot be mixed with --install-parameters-* arguments.". Actually none of those --install-parameters-* is mandatory, see gen_install_parameters in install_param.py