-
Notifications
You must be signed in to change notification settings - Fork 8.1k
Move to 0.17.1 sdk + related fixes #90398
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
Move to 0.17.1 sdk + related fixes #90398
Conversation
This test only works for 64-bit values. On targets which use 32 bits for double, skip this version. The float test will check the 32-bit math for that. Signed-off-by: Keith Packard <[email protected]> Signed-off-by: Anas Nashif <[email protected]>
This file is explicitly included by the espressif hal module. It's an internal file provided by picolibc and newlib. Provide a stub to let code designed for those to work with the minimal C library. Signed-off-by: Keith Packard <[email protected]> Signed-off-by: Anas Nashif <[email protected]>
These two tests are validating that deprecated APIs still work correctly. Which means they will definitely generate compile warnings. To disable -Werror while letting it be enabled for other tests, create a new Kconfig variable, DEPRECATION_TEST and make COMPILER_WARNINGS_AS_ERRORS depend on that option being unselected. Add this option to the two tests so that the resulting configuration disables -Werror. Signed-off-by: Keith Packard <[email protected]> Signed-off-by: Anas Nashif <[email protected]>
Initializing the C++ stack unwinding data structures takes quite a bit of stack space. Increase the TEST_EXTRA_STACK_SIZE when using these. Signed-off-by: Keith Packard <[email protected]>
clang 20 does not like -Ofast. Signed-off-by: Anas Nashif <[email protected]>
ebf1817
to
f9f09a9
Compare
New docker image with SDK 0.17.1 and update docker base is (ubuntu 24.04). Signed-off-by: Anas Nashif <[email protected]>
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.
🎉
f9f09a9
to
e818160
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.
Thanks @nashif and @keith-packard for all the work on this!
test results here https://github.com/zephyrproject-rtos/zephyr-testing/actions/runs/15212241536 and here https://github.com/zephyrproject-rtos/zephyr-testing/actions/runs/15212241536