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

Remove link directives for libutil on linux-gnu targets #2046

Merged
merged 1 commit into from
Jan 28, 2021

Conversation

joshtriplett
Copy link
Member

@joshtriplett joshtriplett commented Jan 28, 2021

These are all handled in src/unix/mod.rs now, which also addresses the
crt-static case; no linux-gnu target should have any link directives in
any other module.

This fixes static linking with glibc for various architectures.

Fixes #2033

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
These are all handled in src/unix/mod.rs now, which also addresses the
crt-static case; no linux-gnu target should have any link directives in
any other module.

This fixes static linking with glibc for various architectures.
@rust-highfive
Copy link

r? @Amanieu

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

@joshtriplett
Copy link
Member Author

r? @JohnTitor

@rust-highfive rust-highfive assigned JohnTitor and unassigned Amanieu Jan 28, 2021
Copy link
Member

@JohnTitor JohnTitor left a comment

Choose a reason for hiding this comment

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

Thanks for the follow-up, Josh!

@JohnTitor
Copy link
Member

@bors r+

@bors
Copy link
Contributor

bors commented Jan 28, 2021

📌 Commit 6859d47 has been approved by JohnTitor

@bors
Copy link
Contributor

bors commented Jan 28, 2021

⌛ Testing commit 6859d47 with merge 80c9e12...

bors added a commit that referenced this pull request Jan 28, 2021
Remove link directives for libutil on linux-gnu targets

These are all handled in src/unix/mod.rs now, which also addresses the
crt-static case; no linux-gnu target should have any link directives in
any other module.

This fixes static linking with glibc for various architectures.
@bors
Copy link
Contributor

bors commented Jan 28, 2021

💔 Test failed - checks-actions

@joshtriplett
Copy link
Member Author

Failure looks like a transient network issue. Retrying.

@bors retry

@bors
Copy link
Contributor

bors commented Jan 28, 2021

⌛ Testing commit 6859d47 with merge 0dce92d...

@joshtriplett
Copy link
Member Author

Looks like we're now hitting rust-lang/cargo#9101 , which was caused by a change in serde.

@joshtriplett
Copy link
Member Author

#2048 should fix the CI issue.

@bors
Copy link
Contributor

bors commented Jan 28, 2021

☀️ Test successful - checks-actions, checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13
Approved by: JohnTitor
Pushing 0dce92d to master...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Some functions result in dynamically linked binaries with +crt-static on non-x86 linux gnu
5 participants