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

Remove non-Library Manager libraries from depends #9

Merged
merged 1 commit into from
Nov 27, 2021
Merged

Remove non-Library Manager libraries from depends #9

merged 1 commit into from
Nov 27, 2021

Conversation

per1234
Copy link
Contributor

@per1234 per1234 commented Nov 18, 2021

The depends field of the library.properties metadata file can be used to provide a list of library dependencies that should be installed in addition during a Library Manager installation. That is the only purpose it serves. It should not contain anything other than names of libraries that are present in the Library Manager index.

Previously, this library's depends field contained elements that were not Library Manager libraries. This had the following effect on installation of the library:

The `depends` field of the library.properties metadata file can be used to provide a list of library dependencies that should be installed in addition during a Library Manager installation. That is the only purpose it serves. It should not contain anything other than names of libraries that are present in the Library Manager index.

Previously, this library's `depends` field contained elements that were not Library Manager libraries. This had the following effect on installation of the library:

- Classic Arduino IDE: the library is installed and the invalid elements ignored
- Arduino IDE 2.x: installation of the library fails silently
- Arduino CLI: installation of the library fails with "Error installing Forced-BME280: No valid dependencies solution found: dependency 'stdint' is not available"
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.

2 participants