Note: there is no decision on whether 1.26.1 should be released yet. This is just a wishlist.
There are a few bad bugs in the 1.26.0 release, and there is a discussion (https://github.com/rust-lang/rust/issues/50646#issuecomment-388596412) about making a point release with a few backports. This issue tracks the PRs we would like to backport, and if we should do the point release at all.
cc @rust-lang/release @rust-lang/core
fn main() -> impl Trait
for non-Termination
traitNaN > NaN
I'd like to propose #50694 for an eventual point release, it makes impl Trait
work with a bunch of common traits that have a Self
default (impl PartialEq
, impl Add
, ...) rather than stack overflow.
It sounds like #50646 has already been fixed in the beta channel. Does the backport require a new PR?
I'd like to strongly vote in favor of a point release containing the fix for #50646.
We should also backport #49523. Right now the distributed source of the 1.26.0 release artifacts don't contain release notes for the 1.26.0 release, which is bad.
We're getting quite a lot of reports of problems on the RLS repo, so I'd be in favour of a point release for the RLS. I'll put together a PR.
I'd also like to update rustfmt due to https://github.com/rust-lang-nursery/rustfmt/issues/2695
PR for RLS/Rustfmt: https://github.com/rust-lang/rust/pull/50794
I nominate #50811 to be included.
Discussed and approved at today's core team meeting. We wanted to include RLS/Rustfmt, termination trait, NaN in consts, and missing manifests (they were all decided either to be high reward or low risk enough to justify inclusion).
@nrc Did #50694 get discussed? It's already accepted and merged for beta, got 3 issue reports and the patch was minor.
Did #50694 get discussed? It's already accepted and merged for beta, got 3 issue reports and the patch was minor.
No. We missed that one. Ping @rust-lang/core - how do we feel about another inclusion?
iiuc it is not a stability concern, just a bug-fix since we currently stack overflow. It looks like a non-trivial patch to me, but I'm not in a great place to judge. Seems like we could live without it.
Added some recent errors to #50646, currently happening on nightly, jic they're of relevance.
The point release PR is up. I believe it includes all items we'd like to; if your desired PR wasn't included please let me know.
Is this testable via rustup or download?
There is no build yet, after the PR will be merged a build will appear in the dev-static bucker.
Release artifacts are built, you can run RUSTUP_DIST_SERVER=https://dev-static.rust-lang.org rustup update stable
to test it locally -- please report problems at https://internals.rust-lang.org/t/rust-1-26-1-prerelease-testing/ or here.
Most helpful comment
Discussed and approved at today's core team meeting. We wanted to include RLS/Rustfmt, termination trait, NaN in consts, and missing manifests (they were all decided either to be high reward or low risk enough to justify inclusion).