-
Notifications
You must be signed in to change notification settings - Fork 239
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Python 3 compatibility #5
base: master
Are you sure you want to change the base?
Conversation
to reflect that this is the Micropython fork
This avoids syntax errors on Python 3, but it doesn't work yet. Main things that need to change is to use bytes/bytearray when working with data. I have a port to Python 3 in the works (tested read/erase/write/verify) but I haven't yet created the commits for it. |
@florisla could you please publish them? |
@florisla I second Kuraga's request to publish a python3 compatible version of this tool as soon as you can. Thanks! |
I also needed Python 3 compatibility, so I fixed it before discovering you were already working on it. I tested read/erase/write/verify myself. Here is a gist. |
I've finally cleaned up my Python 3 branch. It works very similar to @royberkeveld 's of course... |
Please merge changes. +1 for @florisla - i used your version and it works fine with my configuration. |
@susundberg This repository has seen no updates for four years now, I think we can say it's not maintained anymore. You're welcome to use my fork, I'll try to maintain it. |
Should still be backwards compatible with Python 2