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 4: Code-Review+2 Verified+1

(1 comment)

Patchset:

PS4: I think now everything is fine, the check properly detects an invalid combination:

install_cap /home/user/work/simcard_applet_loading/apletts/HelloSTK_09122024.cap --install-parameters 000000 --install-parameters-non-volatile-memory-quota 255 --install-parameters-volatile-memory-quota 255 --install-parameters-stk 010001001505000000000000000000000000 loading cap file: /home/user/work/simcard_applet_loading/apletts/HelloSTK_09122024.cap ... usage: pySim-shell.py FILE [--install-parameters | --install-parameters-*] pySim-shell.py: error: arguments --install-parameters-* are not allowed with --install-parameters

I think this patch is now ready to get merged.