laforge has submitted this change. ( https://gerrit.osmocom.org/c/pysim/+/33424 )
Change subject: pySim-shell: Unregister profile commands during equip ...................................................................... pySim-shell: Unregister profile commands during equip This avoids error messages about re-registering 'AddlShellCommands' commandsets during 'equip()' in the bulk_script command. Change-Id: I893bb5ae95f5c6e4c2be2d133754e427bc92a33d --- M pySim-shell.py 1 file changed, 15 insertions(+), 0 deletions(-) Approvals: laforge: Looks good to me, approved Jenkins Builder: Verified diff --git a/pySim-shell.py b/pySim-shell.py index 3161844..a65e42a 100755 --- a/pySim-shell.py +++ b/pySim-shell.py @@ -215,6 +215,10 @@ if self.rs: lchan = self.rs.lchan[0] lchan.unregister_cmds(self) + if self.rs.profile: + for cmd_set in self.rs.profile.shell_cmdsets: + self.unregister_command_set(cmd_set) + for cmds in [Iso7816Commands, Ts102222Commands, PySimCommands]: cmd_set = self.find_commandsets(cmds) if cmd_set: -- To view, visit https://gerrit.osmocom.org/c/pysim/+/33424 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: I893bb5ae95f5c6e4c2be2d133754e427bc92a33d Gerrit-Change-Number: 33424 Gerrit-PatchSet: 3 Gerrit-Owner: iw0 <osmo...@heychris.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <lafo...@osmocom.org> Gerrit-MessageType: merged