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

Fixes Rainmaker to be a Managed Component #8842

Merged
merged 6 commits into from
Nov 9, 2023

Conversation

SuGlider
Copy link
Collaborator

@SuGlider SuGlider commented Nov 6, 2023

Description of Change

This PR changes CMakLists.txt and idf_component.yml to use most possible Managed Components, including RainMaker.
It also fixes RainMaker Arduino Layer to be compatible with the latest RainMaker release.

Tests scenarios

It has been tested building the Arduino Lib Builder and also by building a RainMaker example using Arduino as IDF component.

Related links

This shall be merged along with espressif/esp32-arduino-lib-builder#145

This changes is targeted to be used within Arduino Lib Builder and with projects that use Arduino as IDF Component.
Fixes RainMaker in order to use it as Managed Component and with the latest RainMaker release.
@SuGlider SuGlider added the 3.0 migration issue relates to migration from 2.X to 3.X version label Nov 6, 2023
@SuGlider SuGlider added this to the 3.0.0 milestone Nov 6, 2023
@SuGlider SuGlider self-assigned this Nov 6, 2023
idf_component.yml Outdated Show resolved Hide resolved
@SuGlider
Copy link
Collaborator Author

SuGlider commented Nov 8, 2023

@me-no-dev - This PR must be merged before fixing the issue in the Lib Builder with sr_models.bin because it adds ESP-SR to the components, which is necessary to run Lib Builder CI correctly in espressif/esp32-arduino-lib-builder#145

@me-no-dev me-no-dev merged commit 1e2ee37 into espressif:master Nov 9, 2023
37 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3.0 migration issue relates to migration from 2.X to 3.X version
Projects
Development

Successfully merging this pull request may close these issues.

2 participants