Hello fixeria, neels, laforge, Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/pysim/+/15504
to look at the new patch set (#17).
Change subject: [DO NOT MERGE YET] Python2 is deprecated, switch to Python3
......................................................................
[DO NOT MERGE YET] Python2 is deprecated, switch to Python3
This also makes sure that the tests are run with python3.
This shound't be merged yet as part of the code still needs
to be converted to python3.
While this seem to work fine with ./pySim-read.py -p 0,
other transports like the the serial and the osmocom
tranports still depend on having python2 string encoding.
This is for instance visible mostly in the PDU encoding
where it depends on ord() which is different between
python2 and python3.
For python2 we have:
ord(c) -> integer
Return the integer ordinal of a one-character string.
whereas for python3 we have:
ord(c, /)
Return the Unicode code point for a one-character string.
Signed-off-by: Denis 'GNUtoo' Carikli <[email protected]>
Change-Id: I430d173535e0cd5bb895b9dfc9070cbc40cfc8ff
---
M README.md
M contrib/jenkins.sh
M pySim-prog.py
M pySim-read.py
M pySim/card_handler.py
M pySim/cards.py
M pySim/commands.py
M pySim/exceptions.py
M pySim/transport/__init__.py
M pySim/transport/calypso.py
M pySim/transport/pcsc.py
M pySim/transport/serial.py
M pySim/ts_51_011.py
M pySim/utils.py
M tests/pysim-test.sh
15 files changed, 21 insertions(+), 25 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/pysim refs/changes/04/15504/17
--
To view, visit https://gerrit.osmocom.org/c/pysim/+/15504
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: I430d173535e0cd5bb895b9dfc9070cbc40cfc8ff
Gerrit-Change-Number: 15504
Gerrit-PatchSet: 17
Gerrit-Owner: gnutoo <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <[email protected]>
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: neels <[email protected]>
Gerrit-MessageType: newpatchset