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 <osmocom(a)heychris.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-MessageType: merged