Skip to content
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

Bump idf-component-manager to 1.5.2 #1321

Merged
merged 1 commit into from Mar 6, 2024
Merged

Conversation

nicklasb
Copy link
Contributor

@nicklasb nicklasb commented Feb 29, 2024

The 1.4.0 of idf-component-manager introduces the "license"-tag which works the same as the library.json "license"-tag.
Several libraries have already started to use this and I just bumped into this with my Robusto library, and when helping Radiolib getting in there. (Edit: saw that arduino-esp32 also use it)

The problem is that libraries before this instead fails, and thus the Platformio build fails, and the recommendation to update to a newer version doesn't help as the old one is installed again in the venv anyway (which actually makes the recommendation sort of pointless and misleading)
I am seeing no issue with going with the latest version here, it is not new either and seems stable.

Temporary workaround:
An ugly one for those that needs this fix before nest platform-espressif-release, at least for *nix variants:
Do the suggested venv idf-component update (not the system one) using sudo, that way the platformio build fails reinstalling the "old" idf-component-manager, but still completes the build process.
sudo -H ~/.platformio/penv/.espidf-5.1.2/bin/python -m pip install --upgrade idf-component-manager
Note that you will have to remove the .espidf-5.1.2 folder to make it reinstall later on

@valeros valeros merged commit 0f5033b into platformio:develop Mar 6, 2024
57 checks passed
@valeros
Copy link
Member

valeros commented Mar 6, 2024

Thanks, merged!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants