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

lib: Bump tokio minimal version to 1.13.0 #320

Merged
merged 2 commits into from Jun 24, 2022

Conversation

jlebon
Copy link
Member

@jlebon jlebon commented Jun 24, 2022

We use Sender::send_replace which was added in 1.13.0:
tokio-rs/tokio#3962

Closes: #316

We use `Sender::send_replace` which was added in 1.13.0:
tokio-rs/tokio#3962

Closes: ostreedev#316
@jlebon
Copy link
Member Author

jlebon commented Jun 24, 2022

Wait, looks like there's more undeclared requirements.

We use `atomic_write` which was added in 0.25:
coreos/cap-std-ext#12
@jlebon
Copy link
Member Author

jlebon commented Jun 24, 2022

OK, got a local build now! I'm not quite sure why CI isn't hitting this. Probably related to the use of Swatinem/rust-cache?

@cgwalters
Copy link
Member

Isn't the problem most likely that you have a local build for which you haven't run cargo update in a while?

@cgwalters cgwalters enabled auto-merge June 24, 2022 20:38
@jlebon
Copy link
Member Author

jlebon commented Jun 24, 2022

Isn't the problem most likely that you have a local build for which you haven't run cargo update in a while?

Oh yeah, that's clearly the immediate cause. I think it's just been a while since I've hacked on a Rust library (since for binaries we have Dependabot hooked up).

@cgwalters
Copy link
Member

OK fun, this one also hit #322

@cgwalters cgwalters disabled auto-merge June 24, 2022 21:02
@cgwalters cgwalters enabled auto-merge June 24, 2022 21:35
@cgwalters cgwalters merged commit 5614f0d into ostreedev:main Jun 24, 2022
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.

Build failures when compiling locally
2 participants