-
Notifications
You must be signed in to change notification settings - Fork 625
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
Upgrade esptool.py #363
Comments
It seems latest master c583756 of esptool works with the rest. |
Well, currently used version of esptool.py also works well. Definitely feel free to upgrade it in your setup. The idea behind esp-open-sdk was to provide conservative set of tools known to work (across years). |
I thing it is too conservative to rely on Python 2.
Jan Pobořil
https://honza.poboril.cz/
12. 5. 2019 v 19:32, Paul Sokolovsky <[email protected]>:
… Well, currently used version of esptool.py also works well. Definitely feel free to upgrade it in your setup. The idea behind esp-open-sdk was to provide conservative set of tools known to work (across years).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.
|
Why suddenly? If you feel so much for Python3, I'd suggest to find a truly worthy project which requires upgrade and help it. My humble suggestion is https://github.com/shedskin/shedskin . |
It is not option for me (if I understand you correctly) because I need esp-open-sdk for building Micropython and it's build failed with Python 2. |
Whose build is failed with Python 2? Both this project and MicroPython is under CI, and there're no known failures. To some up: As someone who works (and even maintains) a lot of projects, I'm great proponent of proverb "Ain't broken ain't fix it" (because I already spend large part of my life on fixing something, and have a loooong list of what's need to be fixed). This ticket remains open to let someone show an issue with current esptool.py and propose a plan of validating any new version (and no worries, I'm subscribed to all bugticket updates for esptool, and what I see doesn't make me want to deal with them, if I have a known working version). |
Quick fix is using shebang |
It think this has become more relevant again now that python 2 is officially no longer supported, and people should have moved to Python 3.5 or newer |
I haven't tried this, but for anyone who needs Python 3 support it should be possible to remove
Not really sure what this means, but recent esptool versions are significantly better tested than v1.2 which had no tests at all. The recent versions also have significantly more active users now (Arduino for ESP8266 and ESP32, platform.io, Micropython, ESP-IDF, etc). Suggest this accounts for any perceived excess of issue reports (many of which turn out to be hardware issues presenting as serial port errors - we're the first port of call for most of those.) |
It means that esptool.py used to be a simple, relatively clean tool, and became a typical "enterprisey" monster. I'm not interested to deal with such in my own free time, in my free time I'm interested to promote cleanliness and simplicity, not support for stuff like serial-over-Internet, which causes issues for some users (abstract example). All in all, I'll probably add a notice that esp-open-sdk is no longer maintained (which is true) and remains a monument of what it was originally intended to be - an example that it's possible to easily build up an open-source toolchain where only warez'ed proprietary toolchains were floating around. |
FWIW: v1.2 (the version distributed with esp-open-sdk) is the version which added that support. The only thing which has changed for that feature in more recent versions is noting the limitations of it. But you are correct that there is more stuff in esptool now compared to back in 2016. Thanks for all your hard work providing an open source toolchain, when such things were much more difficult to come by. |
Currently used version is 3 years old and does not support Python 3. Could we update it?
The text was updated successfully, but these errors were encountered: