smol-rs / async-signal

Asynchronous signal handling
Apache License 2.0
15 stars 6 forks source link

`concurrent-queue` is unavailable for `android` target #16

Closed jvjfk closed 11 months ago

jvjfk commented 11 months ago

concurrent-queue is imported (in Cargo.toml) only if target is linux, but is used when target is linux OR android.

So, async-signal last release (v0.2.1) does not build on android target.

Thank you !

Cargo.toml

[target.'cfg(target_os = "linux")'.dependencies]
concurrent-queue = "2.2.0"

lib.rs

} else if #[cfg(any(target_os = "android", target_os = "linux"))] {
mod signalfd;
use signalfd as sys;
}

signalfd.rs

use concurrent_queue::ConcurrentQueue;
taiki-e commented 11 months ago

Thanks for the report! Filed #18 to fix this.