Skip to content

bootstrap: Update comment in config.library.toml. - #89392

Merged
bors merged 2 commits into
rust-lang:masterfrom
BGR360:master
Oct 1, 2021
Merged

bootstrap: Update comment in config.library.toml.#89392
bors merged 2 commits into
rust-lang:masterfrom
BGR360:master

Conversation

@BGR360

@BGR360BGR360 commented Sep 30, 2021

Copy link
Copy Markdown
Contributor

Downloading LLVM from CI works for all platforms now.

All other templates in this directory already have the proper comment. Seems this one was neglected.

Downloading LLVM from CI works for all platforms now.
@rust-highfive

Copy link
Copy Markdown
Contributor

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @Mark-Simulacrum (or someone else) soon.

Please see the contribution instructions for more information.

@rust-highfiverust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Sep 30, 2021
@Mark-Simulacrum

Copy link
Copy Markdown
Member

@bors r+ rollup

@bors

bors commented Sep 30, 2021

Copy link
Copy Markdown
Collaborator

📌 Commit 1cf905f has been approved by Mark-Simulacrum

@borsbors 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 Sep 30, 2021
ehuss added a commit to ehuss/rust that referenced this pull request Sep 30, 2021
bootstrap: Update comment in config.library.toml.
Downloading LLVM from CI works for all platforms now.
All other templates in this directory already have the proper comment. Seems this one was neglected.
@ehussehuss mentioned this pull request Sep 30, 2021
[llvm]
# Will download LLVM from CI if available on your platform (Linux only for now)
# Will download LLVM from CI if available on your platform.
# https://github.com/rust-lang/rust/issues/77084 tracks support for more platforms

@jyn514jyn514Sep 30, 2021

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hmm, I'm not sure it makes sense to keep this comment now the issue is closed.

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

you're right, i forgot to delete that one. one sec

@BGR360

Copy link
Copy Markdown
ContributorAuthor

ooh, looks like i was just too late and bors beat me to it. do i need to make a new PR for the second commit?

@jyn514

Copy link
Copy Markdown
Member

@BGR360 if #89399 is merged, then yes. Otherwise I can reapprove this PR.

@jyn514

Copy link
Copy Markdown
Member

and yup it failed

@bors r+ rollup

@bors

bors commented Sep 30, 2021

Copy link
Copy Markdown
Collaborator

📌 Commit 098d862 has been approved by jyn514

Manishearth added a commit to Manishearth/rust that referenced this pull request Sep 30, 2021
bootstrap: Update comment in config.library.toml.
Downloading LLVM from CI works for all platforms now.
All other templates in this directory already have the proper comment. Seems this one was neglected.
Manishearth added a commit to Manishearth/rust that referenced this pull request Sep 30, 2021
bootstrap: Update comment in config.library.toml.
Downloading LLVM from CI works for all platforms now.
All other templates in this directory already have the proper comment. Seems this one was neglected.
@ManishearthManishearth mentioned this pull request Oct 1, 2021
bors added a commit to rust-lang-ci/rust that referenced this pull request Oct 1, 2021
…arth
Rollup of 8 pull requests
Successful merges:
- rust-lang#88782 (Fix ICE when `start` lang item has wrong generics)
- rust-lang#89202 (Resolve infered types when complaining about unexpected call type )
- rust-lang#89248 (Suggest similarly named associated items in trait impls)
- rust-lang#89303 (Add `#[must_not_suspend]` to some types in std)
- rust-lang#89306 (thread: implements available_concurrency on haiku)
- rust-lang#89314 (fix(lint): don't suggest refutable patterns to "fix" irrefutable bind)
- rust-lang#89370 (CTFE: tweak aggregate rvalue handling)
- rust-lang#89392 (bootstrap: Update comment in config.library.toml.)
Failed merges:
r? `@ghost`
`@rustbot` modify labels: rollup
@bors
bors merged commit 9ea439d into rust-lang:masterOct 1, 2021
@rustbotrustbot added this to the 1.57.0 milestone Oct 1, 2021
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-borsStatus: 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.

6 participants

@BGR360@rust-highfive@Mark-Simulacrum@bors@jyn514@rustbot