Attention is currently required from: dexter, jolly.
laforge has posted comments on this change by jolly. ( https://gerrit.osmocom.org/c/onomondo-ipa/+/43202?usp=email )
Change subject: Introduce 'operation' option to main file ......................................................................
Patch Set 2:
(1 comment)
File src/ipa/main.c:
https://gerrit.osmocom.org/c/onomondo-ipa/+/43202/comment/2ec3aced_884d10dd?... : PS2, Line 50: printf(" -o get-eim-package ..... Ask eIM for eIM package and process it (default operation)\n"); : printf(" -o add-initial-eim PATH..... set initial eIM configuration from given file\n"); : printf(" -o euicc-memory-reset....... reset eUICC memory\n"); to make it more clear, those should be gruoped somewhere at the beginning or the end of the help message. Just think of how python argparse would sort something like this (see pysim-shell etc for examples).
We might also have the operation as a positional argument without the '-o' after all the actual options. So a bit more like e.g. es90_client.py or es2p_client.py