Skip to content

Fix esp-idf component example build #69

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

Merged
merged 1 commit into from
Jun 4, 2025

Conversation

h2zero
Copy link

@h2zero h2zero commented Jun 4, 2025

Arduino core version was too low for esp-idf versions >= 5.4 due to API changes.

Arduino core version was too low for esp-idf versions >= 5.4 due to API changes.
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR fixes the esp-idf component example build by updating the Arduino core dependency version to meet compatibility requirements for esp-idf versions >= 5.4.

  • The version requirement for espressif/arduino-esp32 is updated from ">=3.0.5" to ">=3.2.0".
Comments suppressed due to low confidence (1)

idf_component_examples/client/main/idf_component.yml:8

  • Consider adding an inline comment explaining that this version bump addresses compatibility issues with esp-idf >= 5.4 due to API changes.
version: ">=3.2.0"

@mathieucarbou mathieucarbou merged commit 725c775 into ESP32Async:main Jun 4, 2025
22 of 28 checks passed
@h2zero
Copy link
Author

h2zero commented Jun 4, 2025

Unfortunately that doesn't fix the CI build for some reason, works locally though 🤷

@mathieucarbou
Copy link
Member

Unfortunately that doesn't fix the CI build for some reason, works locally though 🤷

I am waiting for @me-no-dev to fix it. He knows how. For now, I ignore it.

@me-no-dev
Copy link
Member

CI is fixed in Arduino master, but is not yet released, so CI ends up using core that does not contain the fix, against IDF image that requires it. We are close to pushing 3.2.1, which will fix the issue

@h2zero h2zero deleted the fix-idf-example branch June 4, 2025 20:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants