Open JimGaluska opened 4 years ago
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 https://github.com/chmorgan/esphttpd-freertos/commit/2cd29cb7c25536da93a5258aa3f6414dcabed79b
You could revert to the previous version until we figure out how to fix it: (use --force if git complains)
git checkout 5f449df67140f6e183c465edc8b32e8d399b14e7
git submodule update --init --recursive
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
#!/usr/bin/env python3
to #!/usr/bin/env python
to allow it to build on my msys2 ESP-IDF 3.3.
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: