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

tokio 1.17.0 will not compile on debian stable (bullseye) #4585

Closed
beckerj opened this issue Mar 27, 2022 · 11 comments
Closed

tokio 1.17.0 will not compile on debian stable (bullseye) #4585

beckerj opened this issue Mar 27, 2022 · 11 comments
Labels
A-tokio Area: The main tokio crate C-bug Category: This is a bug.

Comments

@beckerj
Copy link

beckerj commented Mar 27, 2022

Version
tokio v1.17.0

Platform
Linux 5.10.0-12-amd64 #1 SMP Debian 5.10.103-1 (2022-03-07) x86_64 GNU/Linux
Description
tokio 1.17.0 will not compile anymore on debian:
error[E0658]: use of unstable library feature 'renamed_spin_loop'

due to
#4491
because rustc minimum was elevated to 1.49.0 in
#4457

debian comes with 1.48.0+dfsg1-2

I understand that you might not want to fix this because of policies, but it would be nice if you would still support debian stable.

@beckerj beckerj added A-tokio Area: The main tokio crate C-bug Category: This is a bug. labels Mar 27, 2022
@Darksonn
Copy link
Contributor

We have several dependencies that have increased their MSRV to 1.49.0. One of them is parking_lot. I also remember that mio has upgraded, but I can't actually find their MSRV listed anywhere.

@Darksonn
Copy link
Contributor

@asypost It's a legitimate concern that we don't support the rustc on debain stable, but since we have so many dependencies that require 1.49.0, it would probably take a fair amount of work to get all of them to compile on 1.48.0.

@Kixunil
Copy link

Kixunil commented Jun 17, 2022

@asypost there are serious security concerns with that. Rustup doesn't perform signature verification so if their servers or certificates get compromised people are going to install malware. This is not just some theoretical concern, stuff like this already happened, one case was even a popular distro (I think Ubuntu or Mint).

@Darksonn is it feasible to persuade the dependencies to make MSRV Debian-stable-compatible? Also maybe it could still work if people pin them?

@o01eg
Copy link

o01eg commented Jun 17, 2022

What if continue support of tokio 1.16.x for Debian stable?

@Kixunil
Copy link

Kixunil commented Jun 17, 2022

I think a feature would be better, maintaining two separate branches is a huge pain. Especially since I believe currently the only breakage is related to loom which is only needed for development, not for building. I've read the change wrong, sorry.

@Darksonn
Copy link
Contributor

I'm not a maintainer of all dependencies of Tokio, so I can't say for sure what they think about it, but considering its only a single version of difference, it probably isn't impossible to persuade them.

Also maybe it could still work if people pin them?

In the case of parking_lot, there's no 0.12.x version that supports debian stable. There may be other dependencies with the same situation.

@taiki-e
Copy link
Member

taiki-e commented Jul 15, 2022

This seems difficult considering that libs team seems to be actively attempting to drop support for older versions of rustc.
rust-lang/libc#2845

@Darksonn
Copy link
Contributor

Supporting debian stable is not currently a goal, so I will close this. If the situation changes in the future to make it feasible, then I would be happy to accept PRs for that.

@TheBlueMatt
Copy link

This seems difficult considering that libs team seems to be actively attempting to drop support for older versions of rustc.

This doesn't go beyond 1.48, precisely for debian stable :)

@taiki-e
Copy link
Member

taiki-e commented Jan 11, 2023

@TheBlueMatt When I posted that comment, the linked PR proposed increasing the MSRV to 1.56.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-tokio Area: The main tokio crate C-bug Category: This is a bug.
Projects
None yet
Development

No branches or pull requests

7 participants
@o01eg @TheBlueMatt @Darksonn @Kixunil @beckerj @taiki-e and others