-
Notifications
You must be signed in to change notification settings - Fork 67
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add driver
feature
#59
Conversation
764e0d2
to
0774548
Compare
this looks good to me, and is what I'm looking for! (apart from the merge conflict). Is there anything necessary to get this merged apart from a rebase? |
actually I rebased this on master: https://github.com/bpowers/async-io/tree/driver and in local testing, I end up with 2 |
I've been testing a fork of master with @bpowers Is the crates.io version being included as a transitive dependency? edit:
At this point the |
Here's a minimal repro:
With
|
fixes #40
This pull adds a new feature called
driver
. It is enabled by default. If disabledasync-io
will not spawn a background thread to drive the Reactor.Note: afaik adding this feature flag is a breaking change.