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

Enables option split-debuginfo on all platforms #96648

Closed

Conversation

alex-semenyuk
Copy link
Contributor

Closes: #96643

@rust-highfive
Copy link
Collaborator

r? @Mark-Simulacrum

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label May 2, 2022
@bors
Copy link
Contributor

bors commented May 4, 2022

☔ The latest upstream changes (presumably #96695) made this pull request unmergeable. Please resolve the merge conflicts.

@bors
Copy link
Contributor

bors commented May 7, 2022

☔ The latest upstream changes (presumably #96804) made this pull request unmergeable. Please resolve the merge conflicts.

@Mark-Simulacrum
Copy link
Member

I think this will fail on platforms where the flag is already stable, or at least give warnings -- I'm going to go ahead and close in favor of #96758 which should have fixed the known bug for BSD platforms at least. I think it may be worth considering in the future that platform-specific stability is a bad idea here, and instead we should (for example) have stable access through -C... but keep -Z... as unstable across all platforms until all/vast majority of platforms stabilize.

(cc @davidtwco)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Building and testing rustc on *BSD fails on -C split-debuginfo
4 participants