laforge submitted this change.

View Change

Approvals: laforge: Looks good to me, approved Jenkins Builder: Verified
transport/pcsc: throw ReaderError with a message

Before this patch:

$ ./pySim-shell.py -p 0
Card reader initialization failed with an exception of type:
<class 'pySim.exceptions.ReaderError'>

after:

$ ./pySim-shell.py -p 0
Card reader initialization failed with exception:
No reader found for number 0

Change-Id: Id08c4990857f7083a8d1cefc90ff85fc20ab6fef
---
M pySim/transport/pcsc.py
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pySim/transport/pcsc.py b/pySim/transport/pcsc.py
index ca1751a..e3f2546 100644
--- a/pySim/transport/pcsc.py
+++ b/pySim/transport/pcsc.py
@@ -34,7 +34,7 @@
super().__init__(**kwargs)
r = readers()
if reader_number >= len(r):
- raise ReaderError
+ raise ReaderError('No reader found for number %d' % reader_number)
self._reader = r[reader_number]
self._con = self._reader.createConnection()


To view, visit change 27882. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: Id08c4990857f7083a8d1cefc90ff85fc20ab6fef
Gerrit-Change-Number: 27882
Gerrit-PatchSet: 3
Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <pmaier@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-MessageType: merged