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 #19839

Merged
merged 1 commit into from
Jul 20, 2023

Conversation

maribu
Copy link
Member

@maribu maribu commented Jul 20, 2023

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.
@maribu maribu added Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) Area: build system Area: Build system Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Process: needs backport Integration Process: The PR is required to be backported to a release or feature branch labels Jul 20, 2023
@maribu maribu requested a review from benpicco July 20, 2023 08:08
@github-actions github-actions bot added Area: pkg Area: External package ports and removed Area: build system Area: Build system labels Jul 20, 2023
@@ -1,4 +1,3 @@
FEATURES_REQUIRED += arduino
FEATURES_REQUIRED += cpp
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is also technically not needed, as arduino depends on cpp. However, the package uses C++ code itself, (and not only indirectly via the arduino module), so I think we should keep this one.

@riot-ci
Copy link

riot-ci commented Jul 20, 2023

Murdock results

✔️ PASSED

acc1203 pkg/arduino_adafruit_sensor: fix dependencies

Success Failures Total Runtime
7906 0 7906 12m:03s

Artifacts

@maribu
Copy link
Member Author

maribu commented Jul 20, 2023

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 d11255b into RIOT-OS:master Jul 20, 2023
@maribu maribu deleted the arduino-fix-dep branch July 20, 2023 15:49
@maribu
Copy link
Member Author

maribu commented Jul 20, 2023

Backport provided in #19840

@maribu
Copy link
Member Author

maribu commented Jul 20, 2023

Thx :)

bors bot added a commit that referenced this pull request Jul 20, 2023
19840: pkg/arduino_adafruit_sensor: fix dependencies [backport 2023.07] r=benpicco a=maribu

# 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

Co-authored-by: Marian Buschsieweke <marian.buschsieweke@posteo.net>
@MrKevinWeiss MrKevinWeiss added this to the Release 2023.10 milestone Nov 17, 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: needs backport Integration Process: The PR is required to be backported to a release or feature branch 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.

4 participants