smol-rs / async-signal

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

Remove the signalfd backend #20

Closed notgull closed 1 year ago

notgull commented 1 year ago

This backend appears to miss some signals that are raised on other threads. This might be the cause of https://github.com/smol-rs/async-process/issues/55.

I don't feel like kneading signalfd into a better shape, and it probably isn't much of an improvement over the pipe strategy anyways. So I've just removed it.

In addition, this PR also adds tests and removes the libc direct dependency.