espressif / esp-box

The ESP-BOX is a new generation AIoT development platform released by Espressif Systems.
Apache License 2.0
714 stars 173 forks source link

Support to build against ESP-IDF v5.0-dev #18

Closed mahavirj closed 1 year ago

mahavirj commented 2 years ago
alic-maker commented 2 years ago

Thank you for your contribution. We still need more tests for this PR, especially for the speech recognition part, which will take some time. Shanghai is under lockdown, so we can't test it soon 😢

igrr commented 1 year ago

@alic-maker gentle ping, could you please revisit the PR?

I don't think doing complete test should be a blocker for merging this. At the moment, IDF release/v5.0 branch can't be used to build this project, at all. After the PR is merged, the demo app will be at least buildable, so that's already an improvement. If you need to do any fix for the speech recognition part, you can do this in a follow-up MR or PR.

alic-maker commented 1 year ago

Now this mahavirj:feature/support_esp_idf_v5.0-dev branch will fail to compile with IDF release/v5.0. I changed Rainmaker to master, and it seems that the compilation will still fail.

mahavirj commented 1 year ago

Now this mahavirj:feature/support_esp_idf_v5.0-dev branch will fail to compile with IDF release/v5.0. I changed Rainmaker to master, and it seems that the compilation will still fail.

@alic-maker I will update this PR to fix the build problems with latest release/v5.0 branch

mahavirj commented 1 year ago

@alic-maker Please check the latest changes here, PR makes all examples compatible with ESP-IDF 5.x release. Please note that there are still some warnings about deprecated fields but that requires some driver level changes, so we can take them up in followup PRs.

espressif2022 commented 1 year ago

@mahavirj ,hello I tried to merge the RP code and found that the factory_demo could not build successfully;“ERROR: Cannot process component requirements. Multiple candidates to led_strip” I clone the original https://github.com/mahavirj/esp-box.git, branch:feature/support_esp_idf_v5.0-dev, it's the same;

And there are many build warnnings, will there be a follow-up PR to fix this?