-
Notifications
You must be signed in to change notification settings - Fork 2k
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
makefiles/info-global.inc.mk: reset features_required_any #16026
Conversation
1109178
to
8d63bc1
Compare
I think I dealt with the issues and introduced a temporary blacklist for native on the failing flashpage related tests. |
RIOT-OS#15859 exposed a bug in the handling of FEATURES_REQUIRED_ANY, so RIOT-OS#15935 did not run this test, native is currently failing, so blacklist while this is still the case
34205af
to
97b15a6
Compare
# handling of FEATURES_REQUIRED_ANY, so https://github.com/RIOT-OS/RIOT/pull/15935 | ||
# did not run unittests, native is currently failing, so blacklist while this | ||
# is still the case | ||
FEATURES_BLACKLIST += arch_native |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed by #16030
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well the thing is the test is not run without this PR, do you mind removing in your PR?
The test uses module functions, so require it.
97b15a6
to
1d8448d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The build history of this PR shows, that it fixes the issue (and suddenly we have 5000 builds and test runs more ;-)). So ACK.
Contribution description
#15859 added the
newlib
feature, which is added as a dependency for someBOARD
s asFEATURES_REQUIRED_ANY
, this was not cleaned, this made the requirement persist and contaminate the en result. This PR fixes it, but probably some broken tests will show up.Tracking previously uncaught issues:
Testing procedure
This once a RED Murdock should show it.