laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/pysim/+/23533 )
Change subject: filesystem: add new command add_ef ...................................................................... Patch Set 1: I'm sorry, I don't like that approach. I always thought we should support arbitrary files, but not by users creating it on the command line. I think we should try to be more elegant without requiring the user to specify things we can discover. Alternative, for example: Simply allow the user to select any FID (two-digit hex number), and then use the FCP returned by the card to create an EF instance of proper nature (transparent / record / size / ...). Another idea is to have some kind of configuration file (similar to the idea with regard to the ADM/PIN) where users can configure more files. But TBH, I would rather prefer writing proper python code for all the relevant files. I also don't think it's worth investing sysmocom time on this at the moment. -- To view, visit https://gerrit.osmocom.org/c/pysim/+/23533 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: Iafff97443130f8bb8c5bc68f51d2fe1d93fff07c Gerrit-Change-Number: 23533 Gerrit-PatchSet: 1 Gerrit-Owner: dexter <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-CC: laforge <[email protected]> Gerrit-Comment-Date: Mon, 29 Mar 2021 20:34:10 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: No Gerrit-MessageType: comment
