On 09.07.24 08:14, Blair Noctis wrote:
The errors came from the fact that os_pipe is sync, while isahc is async. I've
implemented a simple wrapper for the pipe reader & writer; rather long, thus put
in a fork: https://salsa.debian.org/ncts/rust-isahc/-/commits/replace-sluice.
Note that os_pipe can fail, thus two `todo!()`s. OTOH, sluice returns directly
the pipe, without wrapping in Result, so I think there is space for improvement.
Blair,
thanks for this. Jonas, with this patch at hand, are you ok if I file a
RM request for sluice then ?
isahc is RC-buggy anyway and would need an additional aptch for polling 2.x.
I updated all relevant crates for the Rust team (including zbus and all
rdeps). Once sluice is removed I'd suggest we flip the switch then.
best,
werdahias
