nextest-rs / nextest

A next-generation test runner for Rust.
https://nexte.st
Apache License 2.0
2.2k stars 97 forks source link

android: fails to build with "unresolved import `linux_raw_sys::ioctl`" #862

Closed sylvestre closed 1 year ago

sylvestre commented 1 year ago

Trying to run cargo install nextest on Android, it fails with:

2023-04-29T08:03:36.5411530Z Compiling log v0.4.17
2023-04-29T08:03:36.5412100Z error[E0432]: unresolved import linux_raw_sys::ioctl
2023-04-29T08:03:36.5412710Z --> /data/data/com.termux/files/home/.cargo/registry/src/github.com-1285ae84e5963aae/rustix-0.37.17/src/backend/libc/io/syscalls.rs:384:24
2023-04-29T08:03:36.5413150Z |
2023-04-29T08:03:36.5413830Z 384 | use linux_raw_sys::ioctl::EXT4_IOC_RESIZE_FS;
2023-04-29T08:03:36.5414380Z | could not find ioctl in linux_raw_sys
2023-04-29T08:03:36.5414590Z
2023-04-29T08:03:36.5414840Z For more information about this error, try rustc --explain E0432.
2023-04-29T08:03:36.5415280Z error: could not compile rustix due to previous error
2023-04-29T08:03:36.5415730Z warning: build failed, waiting for other jobs to finish...

Full log can be seen: https://github.com/uutils/coreutils/actions/runs/4837682994/jobs/8621656415#step:8:1767

sunfishcode commented 1 year ago

This is fixed in rustix 0.37.18.

sunshowers commented 1 year ago

Thanks. Going to release a version with the updated rustix shortly.

sunshowers commented 1 year ago

cargo-nextest 0.9.52 just got tagged and will be published within the next 30 minutes once binaries are built. Please test it out and let me know whether it works!

sylvestre commented 1 year ago

Impressive turnaround, bravo :)

sylvestre commented 1 year ago

I can confirm it works great now Thanks again