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

SDK_VERSION: Use Zephyr SDK 0.17.0 #80192

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

stephanosio
Copy link
Member

@stephanosio stephanosio commented Oct 22, 2024

Use Zephyr SDK 0.17.0 in the CI workflows

This series includes the additional patches required to resolve compatibility issues with the SDK 0.17.0 and Picolibc 1.8.8 release.


Includes commits from #79873

Tested in https://github.com/zephyrproject-rtos/zephyr-testing/actions/runs/11455539853

Note that "Run tests with twister" failure is expected in this PR because this workflow runs in the context of the base branch (main).

The Thrift module tests were under the "newlib" namespace, presumably
because newlib was the default at the time of their introduction.

Since newlib is no longer the default libc and Thrift is not being tested
against multiple C libraries, this removes the libc type from the Thrift
test identifiers.

Signed-off-by: Stephanos Ioannidis <root@stephanos.io>
The Thrift library makes use of POSIX C functions such as ctime_r(), which
are not part of the ISO C standard.

This commit adds a Thrift module-wide `_POSIX_C_SOURCE` feature test macro
definition in order to ensure that the required POSIX C functions are
available when compiling the Thrift library.

Note that this was not caught earlier because Newlib and older versions of
Picolibc did not properly fence off some POSIX functions behind the feature
test macros.

Signed-off-by: Stephanos Ioannidis <root@stephanos.io>
This commit increases the test timeout for the wait queue benchmark tests
to 120 seconds because these tests frequently hit the default timeout of 60
seconds during execution.

Signed-off-by: Stephanos Ioannidis <root@stephanos.io>
This commit updates the CI workflows to use the CI image v0.27.3, which
includes Zephyr SDK 0.17.0.

Signed-off-by: Stephanos Ioannidis <root@stephanos.io>
This commit updates ZEPHYR_SDK to point to the Zephyr SDK 0.17.0 release.

Signed-off-by: Stephanos Ioannidis <root@stephanos.io>
@stephanosio stephanosio mentioned this pull request Oct 22, 2024
45 tasks
@stephanosio stephanosio added this to the v4.0.0 milestone Oct 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.