Determine MSRV Policy
yuri-rs opened this issue · comments
I see that current master require 1.65 for 1 let...else statement and 1.64 for few std::future::poll_fn usages.
I don't think it is great, since these are too fresh Rust versions.
Also it would be good to provide clear MSRV guarantees in Readme and include this MSRV in CI.
I think it would be better to revert f6cb0ed at least for the next release
We don't plan on having an MSRV policy right now. This project is pretty early on in its lifecycle and very much not stable yet, so we don't plan on having an MSRV policy until later.
What is the reason that you need a lower MSRV? 1.65 is not very new, given that rust 1.67 is currently the latest one out.
Ok, I see that there is no MSRV policy right now. Just expected some and expected some policy to be aligned with tokio somehow someday.
I, personally, think that MSRV not thought out in general, because it is quite common to just bump MSRV in the project without any indication.
For example, projects with tokio-uring = "0.4.0" dep will not compile on ubuntu-22.04 by rustc right after tokio-uring 0.5.0 with MSRV 1.65 release.
This result in situations like - tokio-rs/tokio#5048
By the way, I don't fully understand why the solution was to copy OneCell into the tokio, rather than specify once_cell = "=1.5.2" in Cargo.
I created this issue because:
- would like to understand MSRV -- understood, no MSRV policy
- feel that having MSRV policy (at least specify MSRV) is good
I think that at least specifying the current MSRV would be good.