Skip to content
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

clean up exposed crate features #50

Merged
merged 1 commit into from
Nov 16, 2023
Merged

clean up exposed crate features #50

merged 1 commit into from
Nov 16, 2023

Conversation

seanmonstar
Copy link
Member

  • Put TokioExecutor and TokioIo into a single rt::tokio module.
  • The rt::tokio module is behind a tokio crate feature.
  • Changed auto crate feature to server-auto.
  • Added client-legacy crate feature, with client::legacy behind it.
  • Removed tcp and runtime features, code now depends on the tokio feature.

@seanmonstar
Copy link
Member Author

@howardjohn I've adjusted some of the features needed to use the legacy client, could you confirm for me that with the changes, your use case works?

@howardjohn
Copy link
Contributor

@howardjohn I've adjusted some of the features needed to use the legacy client, could you confirm for me that with the changes, your use case works?

Thanks! This works for our case, but looks like #49 broke us. With #51 + this PR everything is working fine

- Put `TokioExecutor` and `TokioIo` into a single `rt::tokio` module.
- The `rt::tokio` module is behind a `tokio` crate feature.
- Changed `auto` crate feature to `server-auto`.
- Added `client-legacy` crate feature, with `client::legacy` behind it.
- Removed `tcp` and `runtime` features, code now depends on the `tokio`
  feature.
@seanmonstar seanmonstar merged commit c20cd20 into master Nov 16, 2023
15 checks passed
@seanmonstar seanmonstar deleted the features-cleanup branch November 16, 2023 18:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants