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

use precompiled rustc for non-dist builders #122709

Open
wants to merge 13 commits into
base: master
Choose a base branch
from

Conversation

onur-ozkan
Copy link
Member

@onur-ozkan onur-ozkan commented Mar 18, 2024

Makes non-dist builders to use precompiled CI rustc by default if they are available for the target triple.

As we are going to make rust.download-rustc=if-unchanged default option with #119899, we need to make sure if-unchanged logic never breaks and works as expected.

As an addition, this will significantly improve the build times on CI when there's no change on the compiler.

blocker for #119899

try-job: x86_64-gnu-nopt
try-job: aarch64-apple

@rustbot rustbot added A-testsuite Area: The testsuite used to check the correctness of rustc S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue. labels Mar 18, 2024
@rust-log-analyzer

This comment has been minimized.

@onur-ozkan onur-ozkan force-pushed the use-precompiled-rustc-by-default branch from a080419 to 7a798d5 Compare March 18, 2024 19:21
@rust-log-analyzer

This comment has been minimized.

@bors
Copy link
Contributor

bors commented Mar 18, 2024

☔ The latest upstream changes (presumably #122690) made this pull request unmergeable. Please resolve the merge conflicts.

@onur-ozkan onur-ozkan added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Mar 24, 2024
@onur-ozkan onur-ozkan force-pushed the use-precompiled-rustc-by-default branch 2 times, most recently from 7a8706a to 7ca076d Compare April 1, 2024 13:55
@rust-log-analyzer

This comment has been minimized.

@onur-ozkan onur-ozkan force-pushed the use-precompiled-rustc-by-default branch from 7ca076d to 359b0d8 Compare April 1, 2024 14:09
@rust-log-analyzer

This comment has been minimized.

@rust-log-analyzer

This comment has been minimized.

@onur-ozkan onur-ozkan force-pushed the use-precompiled-rustc-by-default branch from f581ad1 to 6f7cdd6 Compare April 1, 2024 14:45
@rust-log-analyzer

This comment has been minimized.

@rustbot rustbot added the T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) label Apr 2, 2024
@onur-ozkan onur-ozkan force-pushed the use-precompiled-rustc-by-default branch from 8306066 to 795d92b Compare April 2, 2024 15:15
@rust-log-analyzer

This comment has been minimized.

@onur-ozkan onur-ozkan force-pushed the use-precompiled-rustc-by-default branch 2 times, most recently from d7edc43 to 7099a13 Compare April 4, 2024 15:06
@onur-ozkan
Copy link
Member Author

@bors try

bors added a commit to rust-lang-ci/rust that referenced this pull request Apr 4, 2024
…default, r=<try>

prefer precompiled rustc for x86_64-gnu

*-to be filled-*

blocker for rust-lang#119899
r? ghost
@bors
Copy link
Contributor

bors commented Apr 4, 2024

⌛ Trying commit 7099a13 with merge 9ec5616...

@bors
Copy link
Contributor

bors commented Apr 4, 2024

☀️ Try build successful - checks-actions
Build commit: 9ec5616 (9ec5616fdf20bd5f5381043eccffbe26506a80ff)

@onur-ozkan onur-ozkan force-pushed the use-precompiled-rustc-by-default branch from 7099a13 to 731c8eb Compare April 4, 2024 17:31
@onur-ozkan
Copy link
Member Author

@bors try

@bors
Copy link
Contributor

bors commented Apr 4, 2024

⌛ Trying commit 731c8eb with merge 3a8cf09...

bors added a commit to rust-lang-ci/rust that referenced this pull request Apr 4, 2024
…default, r=<try>

prefer precompiled rustc for x86_64-gnu

*-to be filled-*

blocker for rust-lang#119899
r? ghost
@bors bors added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Sep 16, 2024
@onur-ozkan onur-ozkan force-pushed the use-precompiled-rustc-by-default branch from db23d17 to 57274bd Compare September 17, 2024 11:21
@onur-ozkan
Copy link
Member Author

@bors try

@bors
Copy link
Contributor

bors commented Sep 17, 2024

⌛ Trying commit 57274bd with merge 398f406...

bors added a commit to rust-lang-ci/rust that referenced this pull request Sep 17, 2024
…default, r=<try>

use precompiled rustc for non-dist builders

Makes non-dist builders to use precompiled CI rustc by default if they are available for the target triple.

As we are going to make `rust.download-rustc=if-unchanged` default option with rust-lang#119899, we need to make sure `if-unchanged` logic never breaks and works as expected.

As an addition, this will significantly improve the build times on CI when there's no change on the compiler.

blocker for rust-lang#119899

try-job: x86_64-gnu-nopt
@rust-log-analyzer
Copy link
Collaborator

The job x86_64-gnu-nopt failed! Check out the build log: (web) (plain)

Click to see the possible cause of the failure (guessed by this bot)
file:.git/config remote.origin.url=https://github.com/rust-lang-ci/rust
file:.git/config remote.origin.fetch=+refs/heads/*:refs/remotes/origin/*
file:.git/config gc.auto=0
file:.git/config http.https://github.com/.extraheader=AUTHORIZATION: basic ***
file:.git/config branch.try.remote=origin
file:.git/config branch.try.merge=refs/heads/try
file:.git/config submodule.library/backtrace.url=https://github.com/rust-lang/backtrace-rs.git
file:.git/config submodule.library/stdarch.active=true
file:.git/config submodule.library/stdarch.url=https://github.com/rust-lang/stdarch.git
file:.git/config submodule.src/doc/book.active=true
---
---- [ui] tests/ui/allocator/no_std-alloc-error-handler-custom.rs stdout ----

error: test compilation failed although it shouldn't!
status: exit status: 1
command: env -u RUSTC_LOG_COLOR RUSTC_ICE="0" RUST_BACKTRACE="short" "/checkout/obj/build/x86_64-unknown-linux-gnu/stage2/bin/rustc" "/checkout/tests/ui/allocator/no_std-alloc-error-handler-custom.rs" "-Zthreads=1" "-Zsimulate-remapped-rust-src-base=/rustc/FAKE_PREFIX" "-Ztranslate-remapped-path-to-local-path=no" "-Z" "ignore-directory-in-diagnostics-source-blocks=/cargo" "-Z" "ignore-directory-in-diagnostics-source-blocks=/checkout/vendor" "--sysroot" "/checkout/obj/build/x86_64-unknown-linux-gnu/stage2" "--target=x86_64-unknown-linux-gnu" "--check-cfg" "cfg(FALSE)" "--error-format" "json" "--json" "future-incompat" "-Ccodegen-units=1" "-Zui-testing" "-Zdeduplicate-diagnostics=no" "-Zwrite-long-types-to-disk=no" "-Cstrip=debuginfo" "-C" "prefer-dynamic" "-o" "/checkout/obj/build/x86_64-unknown-linux-gnu/test/ui/allocator/no_std-alloc-error-handler-custom/a" "-A" "internal_features" "-Crpath" "-Cdebuginfo=0" "-Lnative=/checkout/obj/build/x86_64-unknown-linux-gnu/native/rust-test-helpers" "-L" "/checkout/obj/build/x86_64-unknown-linux-gnu/test/ui/allocator/no_std-alloc-error-handler-custom/auxiliary" "-C" "panic=abort"
--- stderr -------------------------------
error[E0464]: multiple candidates for `rmeta` dependency `libc` found
##[error]  --> /checkout/tests/ui/allocator/no_std-alloc-error-handler-custom.rs:15:1
   |

@bors
Copy link
Contributor

bors commented Sep 17, 2024

💔 Test failed - checks-actions

Signed-off-by: onur-ozkan <work@onurozkan.dev>
Signed-off-by: onur-ozkan <work@onurozkan.dev>
Signed-off-by: onur-ozkan <work@onurozkan.dev>
Signed-off-by: onur-ozkan <work@onurozkan.dev>
Signed-off-by: onur-ozkan <work@onurozkan.dev>
Signed-off-by: onur-ozkan <work@onurozkan.dev>
Signed-off-by: onur-ozkan <work@onurozkan.dev>
Signed-off-by: onur-ozkan <work@onurozkan.dev>
Signed-off-by: onur-ozkan <work@onurozkan.dev>
Signed-off-by: onur-ozkan <work@onurozkan.dev>
Signed-off-by: onur-ozkan <work@onurozkan.dev>
@onur-ozkan onur-ozkan force-pushed the use-precompiled-rustc-by-default branch from 57274bd to 536ec8e Compare September 17, 2024 16:54
Signed-off-by: onur-ozkan <work@onurozkan.dev>
@onur-ozkan onur-ozkan force-pushed the use-precompiled-rustc-by-default branch from 536ec8e to cdc6642 Compare September 17, 2024 17:00
@onur-ozkan
Copy link
Member Author

@bors try

@bors
Copy link
Contributor

bors commented Sep 17, 2024

⌛ Trying commit cdc6642 with merge 522a6e2...

bors added a commit to rust-lang-ci/rust that referenced this pull request Sep 17, 2024
…default, r=<try>

use precompiled rustc for non-dist builders

Makes non-dist builders to use precompiled CI rustc by default if they are available for the target triple.

As we are going to make `rust.download-rustc=if-unchanged` default option with rust-lang#119899, we need to make sure `if-unchanged` logic never breaks and works as expected.

As an addition, this will significantly improve the build times on CI when there's no change on the compiler.

blocker for rust-lang#119899

try-job: x86_64-gnu-nopt
try-job: aarch64-apple
@onur-ozkan onur-ozkan added S-blocked Status: Marked as blocked ❌ on something else such as an RFC or other implementation work. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Sep 17, 2024
@bors
Copy link
Contributor

bors commented Sep 17, 2024

☀️ Try build successful - checks-actions
Build commit: 522a6e2 (522a6e207e579c9b84b96ae6234cc5026199aac8)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-testsuite Area: The testsuite used to check the correctness of rustc S-blocked Status: Marked as blocked ❌ on something else such as an RFC or other implementation work. T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants