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

Stable backport of #73613 #74746

Merged
merged 3 commits into from
Jul 26, 2020
Merged

Conversation

wesleywiser
Copy link
Member

@wesleywiser wesleywiser commented Jul 25, 2020

This is the backport of #73613 to stable.

r? @ghost

cc @Mark-Simulacrum

In addition the tests added in the original PR passing, I've also confirmed that the test case in #74739 works correctly.

Thus we avoid propagation of a local the moment we encounter references to it.
@wesleywiser wesleywiser changed the title Stable backport of #73669 Stable backport of #73613 Jul 25, 2020
@Mark-Simulacrum
Copy link
Member

@bors r+ p=1000 rollup=never

Thanks! I'll invalidate dev-static once this lands so we can get artifacts for testing.

@bors
Copy link
Contributor

bors commented Jul 25, 2020

📌 Commit 32166ab has been approved by Mark-Simulacrum

@bors bors added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Jul 25, 2020
@bors
Copy link
Contributor

bors commented Jul 25, 2020

⌛ Testing commit 32166ab with merge 9e04db83e57fdce346a7c31587b760e70fe1a538...

@rust-log-analyzer
Copy link
Collaborator

Your PR failed (pretty log, raw log). Through arcane magic we have determined that the following fragments from the build log may contain information about the problem.

Click to expand the log.

I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact @rust-lang/infra. (Feature Requests)

@bors
Copy link
Contributor

bors commented Jul 25, 2020

💔 Test failed - checks-actions

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

Well, that's not good.

2020-07-25T15:49:09.9502009Z error: msys: key "4A6129F4E4B84AE46ED7F635628F528CF3053E04" is unknown
2020-07-25T15:49:13.2340695Z error: key "4A6129F4E4B84AE46ED7F635628F528CF3053E04" could not be looked up remotely

cc @mati865

@mati865
Copy link
Contributor

mati865 commented Jul 25, 2020

I won't have access to any desktop until tomorrow. I think this doesn't happen on nightly right?
If so check if my msys2 PR is present on stable. I'll try be available on zulip right now.

@mati865
Copy link
Contributor

mati865 commented Jul 25, 2020

The PR #73188

@mati865
Copy link
Contributor

mati865 commented Jul 26, 2020

@Mark-Simulacrum I have checked and #73188 is not present on stable. Do you want to backport it or should I search for band-aid workaround?

@Mark-Simulacrum
Copy link
Member

I'm fine with backporting it, will do so sometime today. Thanks!

@Mark-Simulacrum
Copy link
Member

@bors r+ p=1000 rollup=never

@bors
Copy link
Contributor

bors commented Jul 26, 2020

📌 Commit 41895ca has been approved by Mark-Simulacrum

@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 Jul 26, 2020
@bors
Copy link
Contributor

bors commented Jul 26, 2020

⌛ Testing commit 41895ca with merge c367798...

@bors
Copy link
Contributor

bors commented Jul 26, 2020

☀️ Test successful - checks-actions, checks-azure
Approved by: Mark-Simulacrum
Pushing c367798 to stable...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Jul 26, 2020
@bors bors merged commit c367798 into rust-lang:stable Jul 26, 2020
@cuviper cuviper added this to the 1.45 milestone May 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merged-by-bors This PR was explicitly merged by bors. 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