Skip to content

Commit ca8a2e9

Browse files
authored
Unrolled build for #143926
Rollup merge of #143926 - Shourya742:2025-07-14-remove-deprecated-fields, r=jieyouxu Remove deprecated fields in bootstrap This PR removes deprecated fields: 1. `description` - part of rust toml 2. `ccache` - part of llvm toml
2 parents 8c12d76 + 306928f commit ca8a2e9

File tree

4 files changed

+11
-38
lines changed

4 files changed

+11
-38
lines changed

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

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -697,7 +697,7 @@ impl Config {
697697
config.change_id = toml.change_id.inner;
698698

699699
let Build {
700-
mut description,
700+
description,
701701
build,
702702
host,
703703
target,
@@ -749,7 +749,7 @@ impl Config {
749749
compiletest_diff_tool,
750750
compiletest_use_stage0_libtest,
751751
tidy_extra_checks,
752-
mut ccache,
752+
ccache,
753753
exclude,
754754
} = toml.build.unwrap_or_default();
755755

@@ -942,7 +942,7 @@ impl Config {
942942
config.rust_profile_use = flags_rust_profile_use;
943943
config.rust_profile_generate = flags_rust_profile_generate;
944944

945-
config.apply_rust_config(toml.rust, flags_warnings, &mut description);
945+
config.apply_rust_config(toml.rust, flags_warnings);
946946

947947
config.reproducible_artifacts = flags_reproducible_artifact;
948948
config.description = description;
@@ -963,7 +963,7 @@ impl Config {
963963
config.channel = channel;
964964
}
965965

966-
config.apply_llvm_config(toml.llvm, &mut ccache);
966+
config.apply_llvm_config(toml.llvm);
967967

968968
config.apply_gcc_config(toml.gcc);
969969

src/bootstrap/src/core/config/toml/llvm.rs

Lines changed: 1 addition & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,6 @@ define_config! {
1717
tests: Option<bool> = "tests",
1818
enzyme: Option<bool> = "enzyme",
1919
plugins: Option<bool> = "plugins",
20-
// FIXME: Remove this field at Q2 2025, it has been replaced by build.ccache
21-
ccache: Option<StringOrBool> = "ccache",
2220
static_libstdcpp: Option<bool> = "static-libstdcpp",
2321
libzstd: Option<bool> = "libzstd",
2422
ninja: Option<bool> = "ninja",
@@ -97,7 +95,6 @@ pub fn check_incompatible_options_for_ci_llvm(
9795
assertions: _,
9896
tests: _,
9997
plugins,
100-
ccache: _,
10198
static_libstdcpp: _,
10299
libzstd,
103100
ninja: _,
@@ -149,11 +146,7 @@ pub fn check_incompatible_options_for_ci_llvm(
149146
}
150147

151148
impl Config {
152-
pub fn apply_llvm_config(
153-
&mut self,
154-
toml_llvm: Option<Llvm>,
155-
ccache: &mut Option<StringOrBool>,
156-
) {
149+
pub fn apply_llvm_config(&mut self, toml_llvm: Option<Llvm>) {
157150
let mut llvm_tests = None;
158151
let mut llvm_enzyme = None;
159152
let mut llvm_offload = None;
@@ -168,7 +161,6 @@ impl Config {
168161
tests,
169162
enzyme,
170163
plugins,
171-
ccache: llvm_ccache,
172164
static_libstdcpp,
173165
libzstd,
174166
ninja,
@@ -191,13 +183,7 @@ impl Config {
191183
download_ci_llvm,
192184
build_config,
193185
} = llvm;
194-
if llvm_ccache.is_some() {
195-
eprintln!("Warning: llvm.ccache is deprecated. Use build.ccache instead.");
196-
}
197186

198-
if ccache.is_none() {
199-
*ccache = llvm_ccache;
200-
}
201187
set(&mut self.ninja_in_file, ninja);
202188
llvm_tests = tests;
203189
llvm_enzyme = enzyme;

src/bootstrap/src/core/config/toml/rust.rs

Lines changed: 1 addition & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -36,8 +36,6 @@ define_config! {
3636
incremental: Option<bool> = "incremental",
3737
default_linker: Option<String> = "default-linker",
3838
channel: Option<String> = "channel",
39-
// FIXME: Remove this field at Q2 2025, it has been replaced by build.description
40-
description: Option<String> = "description",
4139
musl_root: Option<String> = "musl-root",
4240
rpath: Option<bool> = "rpath",
4341
strip: Option<bool> = "strip",
@@ -320,7 +318,6 @@ pub fn check_incompatible_options_for_ci_rustc(
320318
jemalloc,
321319
rpath,
322320
channel,
323-
description,
324321
default_linker,
325322
std_features,
326323

@@ -388,7 +385,6 @@ pub fn check_incompatible_options_for_ci_rustc(
388385
err!(current_rust_config.std_features, std_features, "rust");
389386

390387
warn!(current_rust_config.channel, channel, "rust");
391-
warn!(current_rust_config.description, description, "rust");
392388

393389
Ok(())
394390
}
@@ -415,12 +411,7 @@ pub(crate) fn validate_codegen_backends(backends: Vec<String>, section: &str) ->
415411
}
416412

417413
impl Config {
418-
pub fn apply_rust_config(
419-
&mut self,
420-
toml_rust: Option<Rust>,
421-
warnings: Warnings,
422-
description: &mut Option<String>,
423-
) {
414+
pub fn apply_rust_config(&mut self, toml_rust: Option<Rust>, warnings: Warnings) {
424415
let mut debug = None;
425416
let mut rustc_debug_assertions = None;
426417
let mut std_debug_assertions = None;
@@ -459,7 +450,6 @@ impl Config {
459450
randomize_layout,
460451
default_linker,
461452
channel: _, // already handled above
462-
description: rust_description,
463453
musl_root,
464454
rpath,
465455
verbose_tests,
@@ -552,14 +542,6 @@ impl Config {
552542
set(&mut self.jemalloc, jemalloc);
553543
set(&mut self.test_compare_mode, test_compare_mode);
554544
set(&mut self.backtrace, backtrace);
555-
if rust_description.is_some() {
556-
eprintln!(
557-
"Warning: rust.description is deprecated. Use build.description instead."
558-
);
559-
}
560-
if description.is_none() {
561-
*description = rust_description;
562-
}
563545
set(&mut self.rust_dist_src, dist_src);
564546
set(&mut self.verbose_tests, verbose_tests);
565547
// in the case "false" is set explicitly, do not overwrite the command line args

src/bootstrap/src/utils/change_tracker.rs

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -481,4 +481,9 @@ pub const CONFIG_CHANGE_HISTORY: &[ChangeInfo] = &[
481481
severity: ChangeSeverity::Warning,
482482
summary: "The current `./x suggest` implementation has been removed due to it being quite broken and a lack of maintenance bandwidth, with no prejudice against re-implementing it in a more maintainable form.",
483483
},
484+
ChangeInfo {
485+
change_id: 143926,
486+
severity: ChangeSeverity::Warning,
487+
summary: "Removed `rust.description` and `llvm.ccache` as it was deprecated in #137723 and #136941 long time ago.",
488+
},
484489
];

0 commit comments

Comments
 (0)