Attention is currently required from: fixeria, laforge.
Hello Jenkins Builder, fixeria, laforge,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/pysim/+/34697?usp=email
to look at the new patch set (#2).
The following approvals got outdated and were removed: Code-Review-1 by fixeria, Verified+1 by Jenkins Builder
Change subject: transport: do not catch exceptions in init_reader ......................................................................
transport: do not catch exceptions in init_reader
We currently catch any exceptions that may occur when the card reader is initialized. Then we print the exception string or the exception type when no string is available. However, a failure during the reader initialization is usually a severe problem, so a traceback would provde a lot of helpful information to debug the issue. So lets not catch any exceptions at this level so that we get the full backtrace.
Related: OS#6210 Change-Id: I4c4807576fe63cf71a7d33b243a3f8fea0b7ff23 --- M pySim-prog.py M pySim-read.py M pySim-shell.py M pySim/transport/__init__.py 4 files changed, 37 insertions(+), 35 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/pysim refs/changes/97/34697/2