Skip to content

Commit

Permalink
Auto merge of rust-lang#134848 - Zalathar:check-compiletest, r=jieyouxu
Browse files Browse the repository at this point in the history
bootstrap: Allow `./x check compiletest`

Did you know that bootstrap didn't support `./x check compiletest`? Well, now it does!

Manually add `"compiletest"` to your `rust-analyzer.check.overrideCommand` check command to get error/warning integration when modifying compiletest.
  • Loading branch information
bors committed Dec 28, 2024
2 parents 8742e05 + c1c2d85 commit b76036c
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 0 deletions.
3 changes: 3 additions & 0 deletions src/bootstrap/src/core/build_steps/check.rs
Original file line number Diff line number Diff line change
Expand Up @@ -501,6 +501,9 @@ tool_check_step!(
);

tool_check_step!(Bootstrap, "bootstrap", "src/bootstrap", SourceType::InTree, false);
// Compiletest is implicitly "checked" when it gets built in order to run tests,
// so this is mainly for people working on compiletest to run locally.
tool_check_step!(Compiletest, "compiletest", "src/tools/compiletest", SourceType::InTree, false);

/// Cargo's output path for the standard library in a given stage, compiled
/// by a particular compiler for the specified target.
Expand Down
1 change: 1 addition & 0 deletions src/bootstrap/src/core/builder/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -905,6 +905,7 @@ impl<'a> Builder<'a> {
check::RustAnalyzer,
check::TestFloatParse,
check::Bootstrap,
check::Compiletest,
),
Kind::Test => describe!(
crate::core::build_steps::toolstate::ToolStateCheck,
Expand Down

0 comments on commit b76036c

Please sign in to comment.