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

CI builds failing due to dependency issues #19

Open
dhalbert opened this issue Feb 18, 2023 · 5 comments
Open

CI builds failing due to dependency issues #19

dhalbert opened this issue Feb 18, 2023 · 5 comments

Comments

@dhalbert
Copy link

arduino-cli lib install "lvgl (=8.2.0)" > /dev/null
attempt 1 failed, 2 retry left
attempt 2 failed, 1 retry left
attempt 3 failed, 0 retry left
FAILED to install dependency lvgl (=8.2.0)

See this
espressif/esp-idf#8495
and maybe newer stuff

@dhalbert dhalbert changed the title CIbuilds failing due to dependency issues CI builds failing due to dependency issues Feb 18, 2023
@caternuson
Copy link
Contributor

This is unfortunately necessary. There's currently an apparent disparity in the version specification syntax between arduino-cli and arduino-lint. Passing an arduino-lint check is needed for the library to be published into the Arduino Library Manager. So that syntax is being used for that reason. As a result, arduino-cli checks will fail.

See also:
#18

@ladyada
Copy link
Member

ladyada commented Feb 18, 2023

@caternuson do we want to manually install the library then... or change ci-arduino's pass/fail critereon?

@caternuson
Copy link
Contributor

I think we're stuck with the current syntax in library.properties for the sake of Arduino Library Manager. So even if the library were manually installed, arduino-cli would probably still balk trying to parse the version syntax.

Crazy hack idea - how about adding a step to this repo's CI, before the pre-install step, that does something like a sed command to directly modify library.properties to change the syntax to what arduino-cli wants? Like a basic search/replace.

@caternuson
Copy link
Contributor

Ex:

sed -i 's/lvgl (=8.2.0)/lvgl@8.2.0/' library.properties

but with whatever $(REPO_FOLDER) prefixes might also be needed

@caternuson
Copy link
Contributor

@dhalbert Was the ESP IDF link just another example of a similar issue? Or is there more tie in from that issue thread that applies here?

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

No branches or pull requests

3 participants