Skip to content

bootstrap: Move several items out of the crate root - #161157

Open
Zalathar wants to merge 8 commits into
rust-lang:mainfrom
Zalathar:bootstrap-root
Open

bootstrap: Move several items out of the crate root#161157
Zalathar wants to merge 8 commits into
rust-lang:mainfrom
Zalathar:bootstrap-root

Conversation

@Zalathar

Copy link
Copy Markdown
Member

Prior to bulk-moving everything out of the crate root, this PR moves several individual items to more suitable modules.

There should be no change to bootstrap behaviour.

This seems to be the module that relies on them the most, and other modules
already pull in things from `dist`.
@rustbot

rustbot commented Aug 16, 2026

Copy link
Copy Markdown
Collaborator

This PR modifies src/bootstrap/src/core/config.

If appropriate, please update CONFIG_CHANGE_HISTORY in src/bootstrap/src/utils/change_tracker.rs.

@rustbot rustbot added A-bootstrap-stamp Area: bootstrap stamp logic 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-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) labels Aug 16, 2026
@rustbot

rustbot commented Aug 16, 2026

Copy link
Copy Markdown
Collaborator

r? @Mark-Simulacrum

rustbot has assigned @Mark-Simulacrum.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: bootstrap
  • bootstrap expanded to 6 candidates
  • Random selection from Mark-Simulacrum, clubby789, jieyouxu

@Zalathar

This comment was marked as resolved.

@Zalathar
Zalathar force-pushed the bootstrap-root branch 2 times, most recently from 6993a01 to 96150c2 Compare August 16, 2026 07:38
@Kobzol

Kobzol commented Aug 16, 2026

Copy link
Copy Markdown
Member

You can r=me once PR CI is green.

r? me

Comment thread src/bootstrap/src/core/compiler.rs Outdated
Comment on lines +9 to +10
/// corresponds to the platform the compiler runs on. This structure is used as
/// a parameter to many methods below.

@fbstj fbstj Aug 16, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

'below' no longer makes sense?

View changes since the review

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

It already doesn’t make much sense as-is, so dropping the sentence entirely is probably wise.

@Zalathar

Copy link
Copy Markdown
Member Author

PR CI is green. I'll push a trivial fix for the comment, then approve.

@Zalathar

Copy link
Copy Markdown
Member Author

@bors r=Kobzol

@rust-bors

rust-bors Bot commented Aug 16, 2026

Copy link
Copy Markdown
Contributor

📌 Commit d5a8464 has been approved by Kobzol

It is now in the queue for this repository.

@rust-bors rust-bors Bot 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 Aug 16, 2026
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Aug 16, 2026
bootstrap: Move several items out of the crate root

Prior to bulk-moving everything out of the crate root, this PR moves several individual items to more suitable modules.

There should be no change to bootstrap behaviour.
GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this pull request Aug 16, 2026
bootstrap: Move several items out of the crate root

Prior to bulk-moving everything out of the crate root, this PR moves several individual items to more suitable modules.

There should be no change to bootstrap behaviour.
rust-bors Bot pushed a commit that referenced this pull request Aug 16, 2026
Rollup of 12 pull requests

Successful merges:

 - #160529 (Upgrade and deduplicate dependencies)
 - #161017 (Library: enforce clippy deref lints in CI)
 - #156674 (bootstrap: add bootstrap step to run stdarch-gen checks in CI)
 - #161006 ([CI] Build newer `binutils` before building `gcc`)
 - #161141 (Add documentation for BPF targets)
 - #161157 (bootstrap: Move several items out of the crate root)
 - #161053 (Add regression test for borrow of array drop type in const)
 - #161073 (Add regression test for path printing with infinitely many visible names)
 - #161099 (Add regression test for unstable def_ident_span fingerprint with incremental recompilation)
 - #161146 (Switch to c8a EC2 runner for auto merges)
 - #161148 ([rustdoc] Put back one removed flaky GUI test (which hopefully isn't flaky anymore))
 - #161150 (add crashtests [4/N])
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Aug 16, 2026
bootstrap: Move several items out of the crate root

Prior to bulk-moving everything out of the crate root, this PR moves several individual items to more suitable modules.

There should be no change to bootstrap behaviour.
rust-bors Bot pushed a commit that referenced this pull request Aug 16, 2026
…uwer

Rollup of 12 pull requests

Successful merges:

 - #160529 (Upgrade and deduplicate dependencies)
 - #161017 (Library: enforce clippy deref lints in CI)
 - #156674 (bootstrap: add bootstrap step to run stdarch-gen checks in CI)
 - #161006 ([CI] Build newer `binutils` before building `gcc`)
 - #161141 (Add documentation for BPF targets)
 - #161157 (bootstrap: Move several items out of the crate root)
 - #161053 (Add regression test for borrow of array drop type in const)
 - #161073 (Add regression test for path printing with infinitely many visible names)
 - #161099 (Add regression test for unstable def_ident_span fingerprint with incremental recompilation)
 - #161103 (cleanup: rip out unnecessary `iter().last()` and `iter().next()`)
 - #161146 (Switch to c8a EC2 runner for auto merges)
 - #161148 ([rustdoc] Put back one removed flaky GUI test (which hopefully isn't flaky anymore))
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Aug 16, 2026
bootstrap: Move several items out of the crate root

Prior to bulk-moving everything out of the crate root, this PR moves several individual items to more suitable modules.

There should be no change to bootstrap behaviour.
rust-bors Bot pushed a commit that referenced this pull request Aug 16, 2026
…uwer

Rollup of 17 pull requests

Successful merges:

 - #160529 (Upgrade and deduplicate dependencies)
 - #161017 (Library: enforce clippy deref lints in CI)
 - #160416 (std: fix unix socket address truncation without a trailing NUL)
 - #161006 ([CI] Build newer `binutils` before building `gcc`)
 - #161141 (Add documentation for BPF targets)
 - #161157 (bootstrap: Move several items out of the crate root)
 - #161185 (std: guard against unwinds in queue-based `Once`)
 - #161186 (miri subtree update)
 - #159855 (std: retry waitid on EINTR in the pidfd wait path)
 - #161053 (Add regression test for borrow of array drop type in const)
 - #161073 (Add regression test for path printing with infinitely many visible names)
 - #161099 (Add regression test for unstable def_ident_span fingerprint with incremental recompilation)
 - #161103 (cleanup: rip out unnecessary `iter().last()` and `iter().next()`)
 - #161136 (Add BPF test for Rust ABI stack arguments)
 - #161146 (Switch to c8a EC2 runner for auto merges)
 - #161148 ([rustdoc] Put back one removed flaky GUI test (which hopefully isn't flaky anymore))
 - #161181 (Add back flaky gui rustdoc test `tests/rustdoc-gui/headers-color.goml`)
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Aug 16, 2026
bootstrap: Move several items out of the crate root

Prior to bulk-moving everything out of the crate root, this PR moves several individual items to more suitable modules.

There should be no change to bootstrap behaviour.
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Aug 16, 2026
bootstrap: Move several items out of the crate root

Prior to bulk-moving everything out of the crate root, this PR moves several individual items to more suitable modules.

There should be no change to bootstrap behaviour.
rust-bors Bot pushed a commit that referenced this pull request Aug 16, 2026
…uwer

Rollup of 17 pull requests

Successful merges:

 - #161017 (Library: enforce clippy deref lints in CI)
 - #160416 (std: fix unix socket address truncation without a trailing NUL)
 - #161006 ([CI] Build newer `binutils` before building `gcc`)
 - #161141 (Add documentation for BPF targets)
 - #161157 (bootstrap: Move several items out of the crate root)
 - #161185 (std: guard against unwinds in queue-based `Once`)
 - #161186 (miri subtree update)
 - #159855 (std: retry waitid on EINTR in the pidfd wait path)
 - #160478 (diagnostics: Suggest fn binding type for unstable closure for<> binders)
 - #161053 (Add regression test for borrow of array drop type in const)
 - #161073 (Add regression test for path printing with infinitely many visible names)
 - #161099 (Add regression test for unstable def_ident_span fingerprint with incremental recompilation)
 - #161103 (cleanup: rip out unnecessary `iter().last()` and `iter().next()`)
 - #161136 (Add BPF test for Rust ABI stack arguments)
 - #161146 (Switch to c8a EC2 runner for auto merges)
 - #161148 ([rustdoc] Put back one removed flaky GUI test (which hopefully isn't flaky anymore))
 - #161181 (Add back flaky gui rustdoc test `tests/rustdoc-gui/headers-color.goml`)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-bootstrap-stamp Area: bootstrap stamp logic A-testsuite Area: The testsuite used to check the correctness of rustc S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants