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

Document unstable --extern options #98226

Merged
merged 1 commit into from
Jun 22, 2022

Conversation

ChrisDenton
Copy link
Member

These are needed for Cargo's build-std feature and for anyone who wanted to do a similar thing outside of Cargo.

@ChrisDenton
Copy link
Member Author

cc @ehuss @jyn514

I'll admit I don't really know what I'm doing but hopefully this is a start.

@jyn514
Copy link
Member

jyn514 commented Jun 18, 2022

Looks like highfive got confused - I'll go ahead and

r? @ehuss

@jyn514 jyn514 added A-resolve Area: Name resolution A-docs Area: documentation for any part of the project, including the compiler, standard library, and tools labels Jun 18, 2022
@jyn514
Copy link
Member

jyn514 commented Jun 18, 2022

cc @petrochenkov

@rust-log-analyzer

This comment has been minimized.

@rust-log-analyzer

This comment has been minimized.

Copy link
Contributor

@ehuss ehuss left a comment

Choose a reason for hiding this comment

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

Thanks! Just a small addition. These really should have tracking issues, but I never got around to it.

@ChrisDenton
Copy link
Member Author

I should probably squish all these commits if that's ok?

Co-Authored-By: Joshua Nelson <[email protected]>
Co-Authored-By: Eric Huss <[email protected]>
@jyn514
Copy link
Member

jyn514 commented Jun 22, 2022

Yeah, I think it would be nice to squash :)

@ehuss
Copy link
Contributor

ehuss commented Jun 22, 2022

Thanks!

@bors r+ rollup

@bors
Copy link
Contributor

bors commented Jun 22, 2022

📌 Commit c20e0e1 has been approved by ehuss

@bors bors added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Jun 22, 2022
bors added a commit to rust-lang-ci/rust that referenced this pull request Jun 22, 2022
Rollup of 10 pull requests

Successful merges:

 - rust-lang#95446 (update CPU usage script)
 - rust-lang#96768 (Use futex based thread parker on Fuchsia.)
 - rust-lang#97454 (Add release notes for 1.62)
 - rust-lang#97516 (clarify how Rust atomics correspond to C++ atomics)
 - rust-lang#97818 (Point at return expression for RPIT-related error)
 - rust-lang#97895 (Simplify `likely!` and `unlikely!` macro)
 - rust-lang#98005 (Add some tests for impossible bounds)
 - rust-lang#98226 (Document unstable `--extern` options)
 - rust-lang#98356 (Add missing period)
 - rust-lang#98363 (remove use of &Alloc in btree tests)

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup
@petrochenkov
Copy link
Contributor

@ChrisDenton
Could you make a tracking issue for this stuff?
(And also link to the item (3) in #93901 (comment) from it.)

@ChrisDenton
Copy link
Member Author

I could but I'd prefer to defer to @ehuss or someone else if they have the time. I don't really know the implementation details or history. But I can do a best effort attempt if it would help get the ball rolling.

@bors bors merged commit 3559718 into rust-lang:master Jun 22, 2022
@rustbot rustbot added this to the 1.63.0 milestone Jun 22, 2022
@ChrisDenton ChrisDenton deleted the doc-extern-options branch June 22, 2022 10:50
@ehuss
Copy link
Contributor

ehuss commented Jun 22, 2022

I'll set up the tracking issues later today.

@ehuss
Copy link
Contributor

ehuss commented Jun 22, 2022

Posted #98401 to add links to the tracking issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-docs Area: documentation for any part of the project, including the compiler, standard library, and tools A-resolve Area: Name resolution S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants