Skip to content

Conversation

dpaoliello
Copy link
Contributor

Fixes #93842

When a raw-dylib is used in a bin crate, we need to link to the library name specified.

@rustbot rustbot added the T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. label Jul 1, 2022
@rust-highfive
Copy link
Contributor

r? @lcnr

(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 Jul 1, 2022
@lcnr
Copy link
Contributor

lcnr commented Jul 4, 2022

r? rust-lang/compiler

@michaelwoerister
Copy link
Member

Thanks for the PR, @dpaoliello! Looks good to me.

@bors r+

@bors
Copy link
Collaborator

bors commented Jul 4, 2022

📌 Commit b8213bb has been approved by michaelwoerister

@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 4, 2022
JohnTitor added a commit to JohnTitor/rust that referenced this pull request Jul 4, 2022
…erister

Enable raw-dylib for bin crates

Fixes rust-lang#93842

When a `raw-dylib` is used in a `bin` crate, we need to link to the library name specified.
Dylan-DPC added a commit to Dylan-DPC/rust that referenced this pull request Jul 4, 2022
…erister

Enable raw-dylib for bin crates

Fixes rust-lang#93842

When a `raw-dylib` is used in a `bin` crate, we need to link to the library name specified.
@matthiaskrgr
Copy link
Member

@bors r- failed in a rollup
#98899 (comment)

@bors bors added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Jul 4, 2022
@dpaoliello
Copy link
Contributor Author

This is the incorrect fix: I need to dig into this more and will provide a new PR once I have the correct fix.

@dpaoliello dpaoliello closed this Jul 5, 2022
@dpaoliello
Copy link
Contributor Author

New PR: !98989

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ICE on #[link(..., kind = "raw-dylib")]
7 participants