-
Notifications
You must be signed in to change notification settings - Fork 7.4k
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
ESP IDF, Arduino as a component, doesn't compile (IDFGH-12940) #9776
Comments
The old build uses mdns build 1.2.1 |
From the log, the error occurs because you are including a C++ header file (WiFi.h) from a C source file (DelishUP-RTOS/components/OTA/OTA.c). |
@igrr Thanks, Now please help with this as well, I have mdns 1.3.1 as a managed component. [1630/1643] Building CXX object esp-idf/OTA/CMakeFiles/__idf_OTA.dir/OTA.cpp.obj
|
Seems like arduino-esp32/idf_component.yml Line 47 in 8b4c130
@meakashrao You can work around the issue by adding |
Thank you, this helps. Deleting and commenting, helped me solve them, is this the right thing to do, doesn't look like it asking for clarity |
I think the latest Arduino-esp32 uses a more recent bug fix release of IDF v5.1.x where these fields/types are introduced. Please try updating your IDF to latest release/v5.1 branch, that should solve the remaining issues. |
This seems to be a mismatch between Arduino Core 3.0.0 and the IDF version it comes with, then. I have the same two issues:
and
The first one is easy to work around by defining I use makeEspArduino, if this makes any difference. |
@everslick which exact version or better commit is used for IDF 5.1? IDF commit from release 3.0.0 can be seen here
|
@meakashrao Your IDF version is to old. |
hmm, I never had to worry about this. I was under the impression, that a specific IDF version comes with a specific CORE version (i.e. they are version pinned to each other). Installing IDF was never a separate thing to do. CORE and IDF came as one. @Jason2866 The contents of the file you posted above is the same as yours, btw.
maybe something is wrong with |
That is true. Are you sure that when you update the core you also run get.py? You must have the wrong libs to get those errors (or you use as IDF component and have the wrong IDF version) |
I just realized that this might be exactly the issue. I DID NOT run get.py again after
|
@everslick with previous versions, the libs used to come inside the core. We changed this for v3.x, so it's usually critical to run get.py when updating the core. |
@meakashrao can we close this issue? |
Thx, @me-no-dev ! Iwill update my build scripts to handle this! BTW, good work on the 3.0.0 release, the migration was pretty straight forward and you managed to keep it pretty API compatible. I'm very impressed! 👍 |
Hey I have just sometime started using IDF, I use IDF in VS Code, what do I need to do? idf.py --version |
@meakashrao get ESP-IDF v5.1.4 instead (or pull branch @everslick glad to hear! I hope the new APIs make more sense in Arduino also :) |
Sure. Thanks will do that |
Answers checklist.
IDF version.
ESP-IDF v5.1.2-666-ga5e3e0b90e
Espressif SoC revision.
ESP32-WROOM-32E-N8
Operating System used.
Windows
How did you build your project?
VS Code IDE
If you are using Windows, please specify command line type.
None
Development Kit.
ESP-WROOM-32
Power Supply used.
USB
What is the expected behavior?
I expected that when using Arduino as a component, the Arduino core to compile, I had use one similar some 3 moths ago and it did compile.
I had to make some changes,
The Arduino Core I downloaded on 6th Feb 2024 did not have this line so I erased it after it threw the error.
Same goes with this, earlier version does not have this and I had to comment this out.
I also tried IDF Version 5.2.1, but it has some other error while downloading mdns as an component, so came back to idf version 5.1.2
What is the actual behavior?
Even after making those above changes, there are some errors in .h files of the Arduino cores.
IDF Bug.zip
Steps to reproduce.
Its a .c file, for uploading here made .txt file
OTA.txt
CMakeLists.txt
Debug Logs.
More Information.
No response
The text was updated successfully, but these errors were encountered: