Attention is currently required from: dexter, fixeria.

laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/pysim/+/35161?usp=email )

Change subject: pySim-shell: Do not use self.lchan.scc when sending raw APDUs.
......................................................................


Patch Set 2:

(1 comment)

File pySim-shell.py:

https://gerrit.osmocom.org/c/pysim/+/35161/comment/c9711334_3094391a
PS2, Line 244: When sending raw APDUs we access the scc object through _scc 
member of the card object. It should also be
             :         # noted that the apdu command plays an exceptional role 
since it is the only card accessing command that
             :         # can be executed without the presence of a runt
I think the improtant bit for the user is not how we access the scc object.  
The important bit is that the 'apdu' command bypasses the currentyl selected 
logical channel.  It might be a good idea to ad this to the docstr of the 
do_apdu method, so it ends up in the user manual.



--
To view, visit https://gerrit.osmocom.org/c/pysim/+/35161?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: I6bbaebe7d7a2013f0ce558ca2da7d58f5e6d991a
Gerrit-Change-Number: 35161
Gerrit-PatchSet: 2
Gerrit-Owner: dexter <pma...@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanits...@sysmocom.de>
Gerrit-Reviewer: laforge <lafo...@osmocom.org>
Gerrit-Attention: fixeria <vyanits...@sysmocom.de>
Gerrit-Attention: dexter <pma...@sysmocom.de>
Gerrit-Comment-Date: Wed, 29 Nov 2023 13:08:33 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment

Reply via email to