-
Notifications
You must be signed in to change notification settings - Fork 13
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
Project does not compile #22
Comments
I can confirm, can no longer build on my setup either. Error is python3 can't be found. @jkent I think we need to support building on any platform/environment that Espressif currently supports, including make on msys2. |
@JimGaluska this was broken in the last commit 2cd29cb You could revert to the previous version until we figure out how to fix it: (use --force if git complains)
|
I forgot about this. Its due to IDF using its packaged version of python 2.7 which doesn't have pathlib. I'll work a version of the script that uses the legacy path functions this afternoon. |
OK, I've fixed the python issue, https://github.com/jkent/esp32-espfs/tree/python-path-compat but there is more to do as we've diverged somewhere... I'll work on it more this evening. |
note I had to change the first line of build-image.py from |
OK, I've got it to build... Looks like the libesphttpd submodule needs to be bumped too. |
You guys are really responsive, Thanks! It gives me confidence to use your library. I have had my attention diverted for the last few days, however I just confirmed that commit #5f449d compiles and works as expected. I also noticed that the commit ID of submodule espfs moved from b87f89a to dd2d57d. The APIs of espfs changed their interface and this change did ripple across to this library. This also would cause a compilation error. @jkent, Is this one of the changes you mentions above? Let me know if it isn't so that I can write this up in a separate issue. |
It was, yes. It didn't help that I haven't had my eyes on this code since October. The API changed to support multiple espfs filesystems. You can have one linked into the binary and one stored in a partition at the same time now. Right now I'm working through a windows build issue, then we will get synced up. |
@phatpaul I think we're better off with Windows now. I do have an issue with mkespfsimage -- I need to port this tool from C to python so we don't have to worry about the user having ming32 or visual studio toolchain around. Its not even possible to build with visual studio because of unistd.h. |
I take that back about porting mkespfsimage... maybe we should just bundle a windows binary. I forgot about heatshrink as a dependency, and while there are bindings, we'd still need a compiler for the bindings. |
I can not get the stock project to compile targeting the ESP32. I am using git commit 2cd29cb of this project and the release/v4.0 branch of the ESP-IDF @ 463a9d8.
Procedure to reproduce:
make
The following error will occur:
The text was updated successfully, but these errors were encountered: