-
Notifications
You must be signed in to change notification settings - Fork 13.3k
Commit esp8266/Arduino@9790e1c python issue #5792
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
Comments
@reaper7, merges automatically close issues here on GH. It's not like Bugzilla or some other tools I've used that require the original submitter to check and sign off. Anyway, as others have pointed out, I re-uploaded the original packaged ZIP and not the one w/the proper subdir structure for Arduino in my rush out the door this AM. D'oh. I have fixed that and verified with my own eyeballs just now with #5797 that python does really expand into tools/python/. Could you please delete your whole tools dir, "git update" to reset everything to fresh checkout state, apply the PR, and give it a go? If not, let's get comments tracked in that PR so we can clean it up. The original esptool.py may have gotten bitten by bit (or brain) rot in the month or so since I stopped looking at it, and all active maintainers are Linux/Mac first (I need to spin up a Win VM w/USB passthru for testing), so Windows' quirks are something of a pain... Thx |
so... #5797 solves problem with python dirs.
|
https://docs.python.org/3/library/os.html#os.fork Windows is a right pain. Python subprocess or multiprocessing should be able to make this work, but I'll need to think about it since I need to push in the pyserial path addition to the code stream. |
@earlephilhower - what is your suggestion, I should temporarily switch to a "stable" 2.5.0b3 version from board manager or wait for a solution for current git version? |
PRs always welcome. :) It's gonna be several hours before I myself will have time to try and fake os.fork() in the uploader script. 2.5.0-rel doesn't have the esptool.py PR, so you can use that. |
Basic Infos
Platform
Settings in IDE
Problem Description
1
After commit 9790e1c , archive python-3.7.2.post1-embed-win32.zip is unpacked into wrong directory.
Should be unpacked to esp8266/tools/python/
but is unpacked to parent dir esp8266/tools/
Compilation break with error -> debug messages below "1. Python is missing"
2
After manually moved all python related files to esp8266/tools/python/ ,
sketch compiles without problems but attempt to load fails with an error -> debug massages below "2. Upload problem"
When I try manual check new included python:
related:
9790e1c#commitcomment-32366937
#5789 FIXED with #5791
MCVE Sketch
examples->digital->BlinkWithoutDelay.ino
Debug Messages
The text was updated successfully, but these errors were encountered: