Skip to content

rename the llvm-tools component to llvm-tools-preview and tweak its image - #51922

Merged
bors merged 2 commits into
rust-lang:masterfrom
japaric:llvm-tools-preview
Jul 1, 2018
Merged

rename the llvm-tools component to llvm-tools-preview and tweak its image#51922
bors merged 2 commits into
rust-lang:masterfrom
japaric:llvm-tools-preview

Conversation

@japaric

Copy link
Copy Markdown
Contributor

@rust-highfiverust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jun 29, 2018
@alexcrichton

Copy link
Copy Markdown
Member

@bors: try

Gonna make sure the artifacts look all good

@bors

bors commented Jun 29, 2018

Copy link
Copy Markdown
Collaborator

⌛ Trying commit c67566a with merge d5a478cf44181eba766bca09d3081db60f74cda8...

@bors

bors commented Jun 29, 2018

Copy link
Copy Markdown
Collaborator

☀️ Test successful - status-travis
State: approved= try=True

@alexcrichton

Copy link
Copy Markdown
Member

Ok that looks alright but I think there's probably a few more locations that need to be renamed? I think that anywhere rls-preview is mentioned you'll want a similar clause for llvm-tools-preview?

@japaric

Copy link
Copy Markdown
ContributorAuthor

@alexcrichton I think that should do it

@alexcrichton

Copy link
Copy Markdown
Member

@bors: r+

@bors

bors commented Jun 30, 2018

Copy link
Copy Markdown
Collaborator

‼️ Invalid head SHA found, retrying: 0000000000000000000000000000000000000000

@bors

bors commented Jun 30, 2018

Copy link
Copy Markdown
Collaborator

📌 Commit 6531879 has been approved by alexcrichton

@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 Jun 30, 2018
emilyalbini added a commit to emilyalbini/rust that referenced this pull request Jul 1, 2018
…richton
rename the llvm-tools component to llvm-tools-preview and tweak its image
as per rust-lang#49584 (comment)
r? @alexcrichton or @Mark-Simulacrum
@emilyalbiniemilyalbini mentioned this pull request Jul 1, 2018
bors added a commit that referenced this pull request Jul 1, 2018
Rollup of 7 pull requests
Successful merges:
- #51511 (Stabilize Iterator::flatten in 1.29, fixes#48213.)
- #51853 (Fix some doc links)
- #51890 (Fix inconsequential typo in GlobalAlloc doc example)
- #51920 (use literal span for concrete type suggestion)
- #51922 (rename the llvm-tools component to llvm-tools-preview and tweak its image)
- #51936 (rename rustc's lld to rust-lld)
- #51961 (Fix typo in /src/librustc_resolve/lib.rs)
Failed merges:
r? @ghost
emilyalbini added a commit to emilyalbini/rust that referenced this pull request Jul 1, 2018
…richton
rename the llvm-tools component to llvm-tools-preview and tweak its image
as per rust-lang#49584 (comment)
r? @alexcrichton or @Mark-Simulacrum
@emilyalbiniemilyalbini mentioned this pull request Jul 1, 2018
bors added a commit that referenced this pull request Jul 1, 2018
Rollup of 7 pull requests
Successful merges:
- #51511 (Stabilize Iterator::flatten in 1.29, fixes#48213.)
- #51853 (Fix some doc links)
- #51890 (Fix inconsequential typo in GlobalAlloc doc example)
- #51920 (use literal span for concrete type suggestion)
- #51921 (improve the error message when `#[panic_implementation]` is missing)
- #51922 (rename the llvm-tools component to llvm-tools-preview and tweak its image)
- #51961 (Fix typo in /src/librustc_resolve/lib.rs)
Failed merges:
r? @ghost
@bors
bors merged commit 6531879 into rust-lang:masterJul 1, 2018
@japaric
japaric deleted the llvm-tools-preview branch July 3, 2018 02:25
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.

4 participants

@japaric@alexcrichton@bors@rust-highfive