-
Notifications
You must be signed in to change notification settings - Fork 12.9k
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
musl: do not compress debug section #59886
Conversation
Old linkers are unable to decompress them and fail to link binaries
@bors: r+ p=1 |
📌 Commit 7276eb5 has been approved by |
I think labels messed up a bit ( |
@rustbot modify labels: +S-waiting-on-bors and -S-waiting-on-review . |
⌛ Testing commit 7276eb5 with merge e586cd9cd1a12154211cde94ec176d425f5e9435... |
💔 Test failed - checks-travis |
The job 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 |
@bors retry |
musl: do not compress debug section This should be beta nominated. Fixes #59411 (this time for real). Test with `DEPLOY=1 ./src/ci/docker/run.sh dist-x86_64-musl`, without `DEPLOY=1` libs are built without debuginfo. r? @alexcrichton
☀️ Test successful - checks-travis, status-appveyor |
[beta] Rollup backports Cherry-picked: * #59886: musl: do not compress debug section * #59891: Fix the link to sort_by_cached_key * #59911: Revert "compile crates under test w/ -Zemit-stack-sizes" * #59978: rustdoc: Remove default keyword from re-exported trait methods * #59989: Fix links to Atomic* in RELEASES.md * #60186: Temporarily accept [i|u][32|size] suffixes on a tuple index and warn * #60309: Add 1.34.1 release notes Rolled up: * #60273: [beta] bootstrap; remove redundant imports. r? @ghost
This should be beta nominated.
Fixes #59411 (this time for real).
Test with
DEPLOY=1 ./src/ci/docker/run.sh dist-x86_64-musl
, withoutDEPLOY=1
libs are built without debuginfo.r? @alexcrichton