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

Explaining how to link mixed C/Rust binaries. #838

Merged
merged 1 commit into from
Nov 12, 2024

Conversation

adetaylor
Copy link
Contributor

Discussed in rust-lang/rust#73632

@ehuss
Copy link
Contributor

ehuss commented Jul 7, 2020

Thanks for the PR! In general, we want the reference to be more about the language, and less about the implementation. To that end, I think this content is better suited for the rustc book. In a sense, almost the entirety of this chapter should probably move, but I am uncertain how exactly that should be structured. So I'm a bit reluctant to continue down the path of adding more compiler-specific information. Maybe you would be interested in figuring out how to untangle this and figure out how to organize things?

@adetaylor
Copy link
Contributor Author

Hi @ehuss, thanks for the comment. I understand this chapter is in a slightly odd status. I'm afraid I'm not likely to be the Untangler here, sorry! Shall I abandon this PR or can I convince you that it doesn't really make things worse?

@traviscross
Copy link
Contributor

traviscross commented Nov 12, 2024

In talking through this on the lang-docs call, we're going to go ahead and merge this so that it lands with wherever the rest of this chapter lands, when we get around to cleaning all this up.

@traviscross traviscross added this pull request to the merge queue Nov 12, 2024
Merged via the queue into rust-lang:master with commit 5110ba4 Nov 12, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants