-
Notifications
You must be signed in to change notification settings - Fork 759
Zephyr: fix the product mini compilation issue #4629
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
base: main
Are you sure you want to change the base?
Zephyr: fix the product mini compilation issue #4629
Conversation
|
If it's alright, could I inquire about your experience with: How to set up CI on a Zephyr platform using GitHub Actions? Since there is no board, a simulation is required. So, which is better: nsim or QEMU? |
|
I @lum1n0us ! Thank you for the quick review. Honestly, I don't have any real experiences for setting a CI on the Zephyr platform using Github Actions. But I may share my insights about native sim vs QEMU. From my understanding about simulation tools:
Also the WAMR project could select a few (embedded) Hardware platforms to support for each of the (RTOS) paltforms supported. It could be aligned with the embedded SIG targeted (hardware) platforms. But in any case at least trying to compile and run the samples should be pretty simple to put in place, and we should begin by that. |
…eep as a temp solution
- Declare it in the api_vm_extension for each platform - Increase the experimental functions doc - Create a os_nanosleep function in a new zephyr_sleep file - add it to the compilation process.
505eb2d to
a3d7396
Compare
lum1n0us
left a comment
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.
Need to resolve this error, reported by CI:
/home/runner/work/wasm-micro-runtime/wasm-micro-runtime/core/shared/platform/common/posix/posix_sleep.c:9:10: fatal error: libc_errno.h: No such file or directory
9 | #include "libc_errno.h"
| ^~~~~~~~~~~~~~
compilation terminated.
This pull request present a fix to #4628.
It introduces the following changes:
os_nanosleepAPI.os_nanosleep.os_APIs inplatform_api_extension.h.simple-fileandsimple-httpsamples compile again.Note
At first I intended to change
os_ioctlreturn type but didn't in the end.