laforge submitted this change.

View Change


Approvals: laforge: Looks good to me, approved Jenkins Builder: Verified
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(-)

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 change 33424. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: I893bb5ae95f5c6e4c2be2d133754e427bc92a33d
Gerrit-Change-Number: 33424
Gerrit-PatchSet: 3
Gerrit-Owner: iw0 <osmocom@heychris.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-MessageType: merged