laforge has uploaded this change for review. ( https://gerrit.osmocom.org/c/pysim/+/27235 ) Change subject: shell: Proper argparser (for help + manual) activate_file ...................................................................... shell: Proper argparser (for help + manual) activate_file Change-Id: I5929ae3deff4d15b5db4a1d866576271c57a955f --- M docs/shell.rst M pySim-shell.py 2 files changed, 11 insertions(+), 4 deletions(-) git pull ssh://gerrit.osmocom.org:29418/pysim refs/changes/35/27235/1 diff --git a/docs/shell.rst b/docs/shell.rst index 58588a2..fcc171b 100644 --- a/docs/shell.rst +++ b/docs/shell.rst @@ -143,7 +143,12 @@ activate_file ~~~~~~~~~~~~~ -Activate the currently selected file. This used to be called REHABILITATE in TS 11.11. +Activate the specified file. This used to be called REHABILITATE in TS 11.11. +You need to specify the name or FID of the file to activate. + +.. argparse:: + :module: pySim-shell + :func: Iso7816Commands.activate_file_parser open_channel ~~~~~~~~~~~~ diff --git a/pySim-shell.py b/pySim-shell.py index ad76ba0..ec417c7 100755 --- a/pySim-shell.py +++ b/pySim-shell.py @@ -805,13 +805,15 @@ self._cmd.poutput("CHV enable successful") def do_deactivate_file(self, opts): - """Deactivate the current EF""" + """Deactivate the currently selected EF""" (data, sw) = self._cmd.card._scc.deactivate_file() + activate_file_parser = argparse.ArgumentParser() + activate_file_parser.add_argument('NAME', type=str, help='File name or FID of file to activate') + @cmd2.with_argparser(activate_file_parser) def do_activate_file(self, opts): """Activate the specified EF""" - path = opts.arg_list[0] - (data, sw) = self._cmd.rs.activate_file(path) + (data, sw) = self._cmd.rs.activate_file(opts.NAME) def complete_activate_file(self, text, line, begidx, endidx) -> List[str]: """Command Line tab completion for ACTIVATE FILE""" -- To view, visit https://gerrit.osmocom.org/c/pysim/+/27235 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: I5929ae3deff4d15b5db4a1d866576271c57a955f Gerrit-Change-Number: 27235 Gerrit-PatchSet: 1 Gerrit-Owner: laforge <laforge(a)osmocom.org> Gerrit-MessageType: newchange