status-im / nim-faststreams

Nearly zero-overhead input/output streams for Nim
Apache License 2.0
125 stars 11 forks source link

remove chronos dependency #38

Closed arnetheduck closed 1 year ago

arnetheduck commented 1 year ago

The chronos backend currently does not work as it relies on nesting waitFor calls, which is not supported by chronos itself - as such, this PR disables the backend in testing and prints a recommendation to users to avoid it until the problem has been resolved.

faststreams used without async continues to work as normal, which also is the most common way to use it - the dependency on chronos in the nimble file is unattractive from a user perspective since it also leads to the the downloading of bearssl and other deps.

Hopefully, by the time the chronos backend can be used, we can also specify conditional dependencies in nimble files.