Skip to content

Commit 18aa419

Browse files
committed
Clarify comment about why bootstrap tests need src/doc/book
1 parent bbe9056 commit 18aa419

File tree

1 file changed

+6
-3
lines changed

1 file changed

+6
-3
lines changed

src/bootstrap/src/core/builder/tests.rs

+6-3
Original file line numberDiff line numberDiff line change
@@ -13,9 +13,12 @@ fn configure_with_args(cmd: &[String], host: &[&str], target: &[&str]) -> Config
1313
config.save_toolstates = None;
1414
config.dry_run = DryRun::SelfCheck;
1515

16-
// Ignore most submodules, since we don't need them for a dry run.
17-
// But make sure to check out the `doc` and `rust-analyzer` submodules, since some steps need them
18-
// just to know which commands to run.
16+
// Ignore most submodules, since we don't need them for a dry run, and the
17+
// tests run much faster without them.
18+
//
19+
// The src/doc/book submodule is needed because TheBook step tries to
20+
// access files even during a dry-run (may want to consider just skipping
21+
// that in a dry run).
1922
let submodule_build = Build::new(Config {
2023
// don't include LLVM, so CI doesn't require ninja/cmake to be installed
2124
rust_codegen_backends: vec![],

0 commit comments

Comments
 (0)