Rust: Rust 1.26.1 point release

Created on 14 May 2018  路  17Comments  路  Source: rust-lang/rust

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


C-feature-request T-core T-release

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).

All 17 comments

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?

49462 affects ARM as well as FreeBSD. I think the point release is warranted.

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

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.

Was this page helpful?
0 / 5 - 0 ratings