Skip to content

Revert "Make libunwind build hermetic" - #76026

Closed
Keruspe wants to merge 1 commit into
rust-lang:masterfrom
Keruspe:revert-72746
Closed

Revert "Make libunwind build hermetic"#76026
Keruspe wants to merge 1 commit into
rust-lang:masterfrom
Keruspe:revert-72746

Conversation

@Keruspe

Copy link
Copy Markdown
Contributor

This reverts commit 21abc88.

Fixes#76020

I'm not sure what exact problem #72746 is supposed to fix, but it's probably incomplete as it breaks the build of 1.46.0 with llvm-libunwind enabled

@rust-highfive

Copy link
Copy Markdown
Contributor

r? @nikomatsakis

(rust_highfive has picked a reviewer for you, use r? to override)

@rust-highfiverust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 28, 2020
@Keruspe

Copy link
Copy Markdown
ContributorAuthor

cc @petrhosek

@Keruspe

Copy link
Copy Markdown
ContributorAuthor

cc @tmandry as you were the one to approve #72746

@jyn514jyn514 added T-libs Relevant to the library team, which will review and decide on the PR/issue. T-libs-api [DEPRECATED; DO NOT USE] labels Sep 15, 2020
@camelidcamelid added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Oct 7, 2020
@Dylan-DPC-zz

Copy link
Copy Markdown

r? @Mark-Simulacrum

@tmandry

Copy link
Copy Markdown
Member

This would end up breaking Fuchsia; #76020 (comment) has more details and talks about how to fix that bug correctly.

@Keruspe

Copy link
Copy Markdown
ContributorAuthor

Let's close this and explore other options such as #77703

@KeruspeKeruspe closed this Oct 8, 2020
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-reviewStatus: Awaiting review from the assignee but also interested parties.T-libsRelevant to the library team, which will review and decide on the PR/issue.T-libs-api[DEPRECATED; DO NOT USE]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1.46.0 fails to build with llvm-libunwind enabled

8 participants

@Keruspe@rust-highfive@Dylan-DPC-zz@tmandry@nikomatsakis@Mark-Simulacrum@jyn514@camelid