Skip to content

Suggest bumping download-ci-llvm-stamp if the build config for llvm changes #112619

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

Merged
merged 1 commit into from
Jun 17, 2023

Conversation

jyn514
Copy link
Member

@jyn514 jyn514 commented Jun 14, 2023

This will hopefully avoid issues like #110474 where the uploaded rust-dev component doesn't match the one you'd get if you built LLVM from source.

cc #110474 (comment)

@rustbot
Copy link
Collaborator

rustbot commented Jun 14, 2023

r? @Mark-Simulacrum

(rustbot has picked a reviewer for you, use r? to override)

@rustbot rustbot added A-meta Area: Issues & PRs about the rust-lang/rust repository itself S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jun 14, 2023
@Mark-Simulacrum
Copy link
Member

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Jun 17, 2023

📌 Commit be9b133 has been approved by Mark-Simulacrum

It is now in the queue for this repository.

@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 Jun 17, 2023
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Jun 17, 2023
…acrum

Suggest bumping download-ci-llvm-stamp if the build config for llvm changes

This will hopefully avoid issues like rust-lang#110474 where the uploaded `rust-dev` component doesn't match the one you'd get if you built LLVM from source.

cc rust-lang#110474 (comment)
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Jun 17, 2023
…acrum

Suggest bumping download-ci-llvm-stamp if the build config for llvm changes

This will hopefully avoid issues like rust-lang#110474 where the uploaded `rust-dev` component doesn't match the one you'd get if you built LLVM from source.

cc rust-lang#110474 (comment)
bors added a commit to rust-lang-ci/rust that referenced this pull request Jun 17, 2023
…iaskrgr

Rollup of 5 pull requests

Successful merges:

 - rust-lang#110805 (Github action to periodically `cargo update` to keep dependencies current)
 - rust-lang#112435 (Allow overwriting the sysroot compile flag via --rustc-args)
 - rust-lang#112610 (Bump stdarch)
 - rust-lang#112619 (Suggest bumping download-ci-llvm-stamp if the build config for llvm changes)
 - rust-lang#112738 (make ice msg "Unknown runtime phase" a bit nicer)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 27af5ed into rust-lang:master Jun 17, 2023
@rustbot rustbot added this to the 1.72.0 milestone Jun 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-meta Area: Issues & PRs about the rust-lang/rust repository itself 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.

4 participants