Make async-std dependency optional about async-native-tls HOT 6 CLOSED

the current version does not use the async-std runtime, it only uses some primitives of it, so it should be runtime agnostic already. I haven’t tested it though

from async-native-tls.

The problem is that it (non-optionally) depends on async-std and brings in a lot of unneeded code (and compile-time) if using it with a different async runtime. Ideally instead of async-std, it would only use whatever is available from the futures crate.

Is there anything used here that is only available in async-std?

from async-native-tls.

Is there anything used here that is only available in async-std?

yes: https://github.com/async-email/async-native-tls/blob/master/src/acceptor.rs#L60-L61

from async-native-tls.

I see. That makes it rather tricky to make the whole things runtime-independent as you actually depend on async-std :)

from async-native-tls.

Yeah, I guess it could be behind a default feature flag, and the exposed API that is runtime independent could be smaller

from async-native-tls.

Maybe that solves it? #14

from async-native-tls.