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

Upgrade to tokio 0.3.6 from crates.io #1583

Merged
merged 2 commits into from
Jan 12, 2021

Conversation

teor2345
Copy link
Contributor

@teor2345 teor2345 commented Jan 12, 2021

Motivation

Zebra depends on a git version of tokio, but tokio has released 0.3.6, which contains the panic fixes we need.

Solution

The code in this pull request:

  • Passes existing tests

Review

Anyone can review this change, but @dconnolly manages dependencies.

Let's get this change merged this week, so we can test it in the later half of the sprint.

Related Issues

Closes #1539

Follow Up Work

Remove git dependencies on Hyper, Metrics, Tower #1086

  • waiting on stable releases of metrics and metrics-http-exporter that depend on tokio 1.0

And remove the tokio git dependency patch
We're now targeting tokio 1.0, because some dependencies did not release
a stable version that depends on tokio 0.3
@teor2345 teor2345 added A-dependencies Area: Dependency file updates C-cleanup Category: This is a cleanup labels Jan 12, 2021
@teor2345 teor2345 added this to the 2021 Sprint 1 milestone Jan 12, 2021
@teor2345 teor2345 requested review from dconnolly and yaahc January 12, 2021 08:05
@teor2345 teor2345 self-assigned this Jan 12, 2021
@mpguerra mpguerra removed this from the 2021 Sprint 1 milestone Jan 12, 2021
@dconnolly dconnolly merged commit ba5654d into ZcashFoundation:main Jan 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-dependencies Area: Dependency file updates C-cleanup Category: This is a cleanup
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update to tokio 0.3.6 and remove git dependency
3 participants