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

1.29.2 stable point release #54808

Merged
merged 5 commits into from
Oct 6, 2018
Merged

Conversation

pietroalbini
Copy link
Member

This point release includes a backport of #54639 (a miscompilation) and the fix for #54206 (rls missing on windows-gnu). It also backports a release notes fix (#54150).

The target date for the release is Thursday 11th.

r? @Mark-Simulacrum
cc @rust-lang/core @rust-lang/release

@rust-highfive
Copy link
Collaborator

⚠️ Warning ⚠️

  • Pull requests are usually filed against the master branch for this repo, but this one is against stable. Please double check that you specified the right target!

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Oct 4, 2018
RELEASES.md Outdated
Version 1.29.2 (2018-10-11)
===========================

Compatibility Notes
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this the right category for these fixes? They don't look like they break backwards compatibility.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Other point releases did put all of the fixes in the compatibility notes. I just removed the section name.

RELEASES.md Outdated
Compatibility Notes
-------------------

- [Fixed unsoundness in the aliasing rules, which caused miscompilation.][54639]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd suggest something like "Work around aliasing-related LLVM bug, which caused miscompilations" here. The current description sounds to me like there was some kind of unsoundness in Rust's aliasing rules.

@Mark-Simulacrum
Copy link
Member

This all looks good to me, I'd like to run the RLS fixing change by @alexcrichton to make sure this is exactly what you wanted (since you had some concerns earlier, I believe). So, r? @alexcrichton for final approval

@alexcrichton
Copy link
Member

@bors: r+ p=100

@bors
Copy link
Contributor

bors commented Oct 5, 2018

📌 Commit a154257 has been approved by alexcrichton

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Oct 5, 2018
@alexcrichton
Copy link
Member

I think this is the right fix, but I'll confirm with the actual artifacts once they exist.

@bors
Copy link
Contributor

bors commented Oct 5, 2018

⌛ Testing commit a154257 with merge 17a9dc7...

bors added a commit that referenced this pull request Oct 5, 2018
1.29.2 stable point release

This point release includes a backport of #54639 (a miscompilation) and the fix for #54206 (rls missing on windows-gnu). It also backports a release notes fix (#54150).

The target date for the release is Thursday 11th.

r? @Mark-Simulacrum
cc @rust-lang/core @rust-lang/release
@bors
Copy link
Contributor

bors commented Oct 6, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing 17a9dc7 to stable...

@bors bors merged commit a154257 into rust-lang:stable Oct 6, 2018
@alexcrichton
Copy link
Member

Cargo seems functional in 1.29.2 and the RLS was built for x86_64-pc-windows-gnu, should be good to go!

@pietroalbini pietroalbini deleted the stable-1.29.2 branch October 6, 2018 10:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants