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

Add documentation for aarch64-apple-ios-sim target #85781

Merged
merged 2 commits into from
May 30, 2021

Conversation

badboy
Copy link
Member

@badboy badboy commented May 28, 2021

Documentation as requested for MCP 428 to promote this target to Tier 2.

Currently it calls out that it's Tier 3. That should be changed if this target is promoted, but this PR could also land before that.

Note: probably should get signoff from the compiler team based on that MCP.

@rust-highfive
Copy link
Collaborator

r? @GuillaumeGomez

(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 May 28, 2021
@GuillaumeGomez
Copy link
Member

r? @Amanieu

@badboy
Copy link
Member Author

badboy commented May 28, 2021

Oh, I just realized that the "Building Rust programs" are not valid as of right now, because target artifacts are not shipped. So right now users would need nightly + -Z build-std for that.
If wanted I can make that more clear there.

@GuillaumeGomez
Copy link
Member

Please do so then. There is never too much clarity. :)

@rust-log-analyzer

This comment has been minimized.

@badboy
Copy link
Member Author

badboy commented May 28, 2021

Hm, I could fix the tier-check to also consider linked targets (which means hackily parsing markdown) or remove the link.

@badboy
Copy link
Member Author

badboy commented May 28, 2021

Oh, actually, it's easier than that: We can just check for '`' somewhere in the line

@badboy badboy force-pushed the document-aarch-ios-sim-support branch from 7bf245b to 1f3539b Compare May 28, 2021 13:37
@rust-log-analyzer

This comment has been minimized.

@badboy badboy force-pushed the document-aarch-ios-sim-support branch from 1f3539b to aa1b127 Compare May 29, 2021 12:43
@Amanieu
Copy link
Member

Amanieu commented May 30, 2021

@bors r+ rollup

@bors
Copy link
Contributor

bors commented May 30, 2021

📌 Commit aa1b127 has been approved by Amanieu

@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 May 30, 2021
Dylan-DPC-zz pushed a commit to Dylan-DPC-zz/rust that referenced this pull request May 30, 2021
…rt, r=Amanieu

Add documentation for aarch64-apple-ios-sim target

Documentation as requested for [MCP 428](rust-lang/compiler-team#428) to promote this target to Tier 2.

Currently it calls out that it's Tier 3. That should be changed if this target is promoted, but this PR could also land before that.

Note: probably should get signoff from the compiler team based on that MCP.
bors added a commit to rust-lang-ci/rust that referenced this pull request May 30, 2021
…laumeGomez

Rollup of 8 pull requests

Successful merges:

 - rust-lang#85285 (Add eslint checks to CI)
 - rust-lang#85709 (Use correct edition when parsing `:pat` matchers)
 - rust-lang#85762 (Do not try to build LLVM with Zlib on Windows)
 - rust-lang#85770 (Remove `--print unversioned-files` from rustdoc )
 - rust-lang#85781 (Add documentation for aarch64-apple-ios-sim target)
 - rust-lang#85801 (Add `String::extend_from_within`)
 - rust-lang#85817 (Fix a typo)
 - rust-lang#85818 (Don't drop `PResult` without handling the error)

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 2d30bc7 into rust-lang:master May 30, 2021
@rustbot rustbot added this to the 1.54.0 milestone May 30, 2021
@badboy badboy deleted the document-aarch-ios-sim-support branch May 31, 2021 07:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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