Attention is currently required from: jolly.

dexter has posted comments on this change by jolly. ( 
https://gerrit.osmocom.org/c/onomondo-ipa/+/43042?usp=email )

Change subject: V1.2: Add function ES10b.ReturnFromFallback
......................................................................


Patch Set 3:

(1 comment)

File src/ipa/main.c:

https://gerrit.osmocom.org/c/onomondo-ipa/+/43042/comment/0e533f57_d41bf472?usp=email
 :
PS3, Line 42: tatic void print_help(void)
            : {
            :   printf("options:\n");
            :   printf(" -h ......................... print this text.\n");
            :   printf(" -t TAC ..................... set TAC (default: %s)\n", 
DEFAULT_TAC);
            :   printf(" -e eimId ................... set preferred eIM (in 
case the eUICC has multiple)\n");
            :   printf(" -r N ....................... set reader number 
(default: %d)\n", DEFAULT_READER_NUMBER);
            :   printf(" -c N ....................... set logical channel 
number (default: %d)\n", DEFAULT_CHANNEL_NUMBER);
            :   printf(" -f PATH .................... set initial eIM 
configuration\n");
            :   printf(" -m ......................... reset eUICC memory\n");
            :   printf(" --refresh-flag.............. Make eUICC send a CAT 
refresh after switching profile\n");
            :   printf(" --execute-fallback-mechanism Trigger enabling of 
fallback profile\n");
            :   printf(" --return-from-fallback...... Trigger return from 
fallback profile to previous profile\n");
            :   printf(" -n PATH .................... path to nvstate file 
(default: %s)\n", DEFAULT_NVSTATE_PATH);
            :   printf(" -y NUM ..................... number of retries for 
ESipa requests (default: %u)\n",
            :          DEFAULT_ESIPA_REQ_RETRIES);
            :   printf(" -a ......................... ask end user for 
consent\n");
            :   printf(" -C ......................... CA (Certificate 
Authority) Bundle file\n");
            :   printf(" -S ......................... disable HTTPS\n");
            :   printf(" -I ......................... disable SSL certificate 
verification (insecure)\n");
            :   printf(" -E ......................... emulate IoT eUICC 
(compatibility mode to use consumer eUICCs)\n");
            :   printf(" -1 ......................... force the IPAd to process 
only one eUICC package (debug, use with caution)\n");
            : }
> I have a general problem with this flat approach of options, mixing actual 
> flags/options (like -t/-e […]
Makes sense, the -m, -f are also operations, this should be re-aligned too.

So lets see what operations we have:

get-eim-package (normal)
add-initial-eim (currently -f)
euicc-memory-reset (currently -m)
execute-fallback-mechanism
return-from-fallback



--
To view, visit https://gerrit.osmocom.org/c/onomondo-ipa/+/43042?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings?usp=email

Gerrit-MessageType: comment
Gerrit-Project: onomondo-ipa
Gerrit-Branch: master
Gerrit-Change-Id: Ib49a8ef3a5ea5639d881ca51eaeeea24f083fe0e
Gerrit-Change-Number: 43042
Gerrit-PatchSet: 3
Gerrit-Owner: jolly <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <[email protected]>
Gerrit-CC: laforge <[email protected]>
Gerrit-Attention: jolly <[email protected]>
Gerrit-Comment-Date: Fri, 07 Aug 2026 15:35:32 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: laforge <[email protected]>

Reply via email to