Skip to content

Commit 13e16a9

Browse files
committed
use Config::get_builder_toml for ci-rustc config parsing
Signed-off-by: onur-ozkan <[email protected]>
1 parent 018ed9a commit 13e16a9

File tree

1 file changed

+3
-5
lines changed

1 file changed

+3
-5
lines changed

src/bootstrap/src/core/config/config.rs

+3-5
Original file line numberDiff line numberDiff line change
@@ -2355,18 +2355,15 @@ impl Config {
23552355
self.download_ci_rustc(commit);
23562356

23572357
if let Some(config_path) = &self.config {
2358-
let builder_config_path =
2359-
self.out.join(self.build.triple).join("ci-rustc").join(BUILDER_CONFIG_FILENAME);
2360-
2361-
let ci_config_toml = match Self::get_toml(&builder_config_path) {
2358+
let ci_config_toml = match self.get_builder_toml("ci-rustc") {
23622359
Ok(ci_config_toml) => ci_config_toml,
23632360
Err(e) if e.to_string().contains("unknown field") => {
23642361
println!("WARNING: CI rustc has some fields that are no longer supported in bootstrap; download-rustc will be disabled.");
23652362
println!("HELP: Consider rebasing to a newer commit if available.");
23662363
return None;
23672364
},
23682365
Err(e) => {
2369-
eprintln!("ERROR: Failed to parse CI rustc config at '{}': {e}", builder_config_path.display());
2366+
eprintln!("ERROR: Failed to parse CI rustc config.toml: {e}");
23702367
exit!(2);
23712368
},
23722369
};
@@ -2829,6 +2826,7 @@ impl Config {
28292826

28302827
/// Compares the current `Llvm` options against those in the CI LLVM builder and detects any incompatible options.
28312828
/// It does this by destructuring the `Llvm` instance to make sure every `Llvm` field is covered and not missing.
2829+
#[cfg(not(feature = "bootstrap-self-test"))]
28322830
pub(crate) fn check_incompatible_options_for_ci_llvm(
28332831
current_config_toml: TomlConfig,
28342832
ci_config_toml: TomlConfig,

0 commit comments

Comments
 (0)