fix(build-analysis): partially reverted b43404d

Remove HTML timing output generation
in favor of the upcoming JSONL logging system.

Author: weihanglo

src/cargo/core/compiler/build_config.rs

@@ -115,18 +115,6 @@ impl BuildConfig {
             (None, _) => false,
         };
 
-        let timing_outputs = match (cfg.analysis.as_ref(), gctx.cli_unstable().build_analysis) {
-            // Enable HTML output to pretend we are persisting timing data for now.
-            (Some(analysis), true) if analysis.enabled => vec![TimingOutput::Html],
-            (Some(_), false) => {
-                gctx.shell().warn(
-                    "ignoring 'build.analysis' config, pass `-Zbuild-analysis` to enable it",
-                )?;
-                Vec::new()
-            }
-            _ => Vec::new(),
-        };
-
         Ok(BuildConfig {
             requested_kinds,
             jobs,
@@ -142,7 +130,7 @@ impl BuildConfig {
             rustfix_diagnostic_server: Rc::new(RefCell::new(None)),
             export_dir: None,
             future_incompat_report: false,
-            timing_outputs,
+            timing_outputs: Vec::new(),
             sbom,
             compile_time_deps_only: false,
         })

tests/testsuite/build_analysis.rs

@@ -37,7 +37,6 @@ fn simple() {
         .masquerade_as_nightly_cargo(&["build-analysis"])
         .with_stderr_data(str![[r#"
 [CHECKING] foo v0.0.0 ([ROOT]/foo)
-      Timing report saved to [ROOT]/foo/target/cargo-timings/cargo-timing-[..].html
 [FINISHED] `dev` profile [unoptimized + debuginfo] target(s) in [ELAPSED]s
 
 "#]])