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

pkg/arduino_adafruit_sensor: fix dependencies [backport 2023.07] #19840

Merged

Conversation

maribu
Copy link
Member

@maribu maribu commented Jul 20, 2023

Backport of #19839

Contribution description

This fixes the dependencies of the arduino_adafruit_sensor package, which previously relied on the arduino feature. This feature no longer exists, as it was split into more fine granular features. However, the module should never have used that feature directly in the first place, but rather just use the arduino module. This in turn depends on the correct features.

Testing procedure

tests/arduino_adafruit_sensor should again be supported by boards that have the features required by the arduino module.

Issues/PRs references

Fallout of #19759

This fixes the dependencies of the `arduino_adafruit_sensor` package,
which previously relied on the `arduino` feature. This feature no longer
exists, as it was split into more fine granular features. However, the
module should never have used that feature directly in the first place,
but rather just use the arduino module. This in turn depends on the
correct features.

(cherry picked from commit acc1203)
@maribu maribu added Area: pkg Area: External package ports CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer Process: release backport Integration Process: The PR is a release backport of a change previously provided to master Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) labels Jul 20, 2023
@riot-ci
Copy link

riot-ci commented Jul 20, 2023

Murdock results

✔️ PASSED

ff26c5c pkg/arduino_adafruit_sensor: fix dependencies

Success Failures Total Runtime
6941 0 6941 10m:46s

Artifacts

@benpicco
Copy link
Contributor

bors merge

@bors
Copy link
Contributor

bors bot commented Jul 20, 2023

Build succeeded!

The publicly hosted instance of bors-ng is deprecated and will go away soon.

If you want to self-host your own instance, instructions are here.
For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.

@bors bors bot merged commit ca4e4ec into RIOT-OS:2023.07-branch Jul 20, 2023
@maribu maribu deleted the backport/2023.07/arduino-fix-dep branch July 20, 2023 21:11
@benpicco benpicco added this to the Release 2023.07 milestone Aug 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: pkg Area: External package ports CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer Process: release backport Integration Process: The PR is a release backport of a change previously provided to master Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants