-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
sys_util: enable build for non-musl libraries #639
Conversation
Related to: #610 |
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.
Tested that this enables building on Amazon Linux 2's Rust toolchain (which uses glibc). 👍
519a0d2
The `ioctl` function signature in the musl library differs from that of glibc by having a different type for the request parameter. Thus, this fix enables successful build on other non-musl libraries using a `#cfg` macro. Signed-off-by: Diana Popa <dpopa@amazon.com>
When compiling with GNU there is at least one violation of the seccomp filter: signal: 31, SIGSYS: bad system call. This violation is causing cargo test to exit with error. Signed-off-by: Andreea Florescu <fandree@amazon.com>
Signed-off-by: Diana Popa <dpopa@amazon.com>
Adds a integration tests for ensuring correctness of the doc tests. The doc tests can only be run by using the default host target (i.e GNU). Signed-off-by: Diana Popa <dpopa@amazon.com>
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.
This PR had 3 approvals before the rebase, approving now for the merge.
The
ioctl
function signature in the musl library (i.e musl ioctl.c) differs from that ofglibc (i.e. glibc's ioctl) by having a different type for the request parameter. Thus,
this fix enables successful build on other non-musl libraries using
a
#cfg
macro.How to test this?