Skip to content

Commit

Permalink
Rollup merge of #134331 - RalfJung:bootstrap-error-index, r=onur-ozkan
Browse files Browse the repository at this point in the history
bootstrap: make ./x test error-index work

This makes it more likely that someone who sees an error index CI failure will be able to figure out how to reproduce that locally. Note that bootstrap already prints "Testing stage2 error-index", which is misleading since the test is actually called error_index_generator.
  • Loading branch information
jieyouxu authored Dec 15, 2024
2 parents 54e2d39 + 4795399 commit b5602cf
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion src/bootstrap/src/core/build_steps/test.rs
Original file line number Diff line number Diff line change
Expand Up @@ -2442,7 +2442,9 @@ impl Step for ErrorIndex {
const ONLY_HOSTS: bool = true;

fn should_run(run: ShouldRun<'_>) -> ShouldRun<'_> {
run.path("src/tools/error_index_generator")
// Also add `error-index` here since that is what appears in the error message
// when this fails.
run.path("src/tools/error_index_generator").alias("error-index")
}

fn make_run(run: RunConfig<'_>) {
Expand Down

0 comments on commit b5602cf

Please sign in to comment.