Skip to content

Commit

Permalink
Revert "Do not cache rustc info in deps resolver (#3107)"
Browse files Browse the repository at this point in the history
This reverts commit 834df35.
  • Loading branch information
havasd committed Dec 16, 2024
1 parent 8f39cc1 commit c3f6b84
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 16 deletions.
1 change: 0 additions & 1 deletion crate_universe/src/metadata/cargo_tree_resolver.rs
Original file line number Diff line number Diff line change
Expand Up @@ -150,7 +150,6 @@ impl TreeResolver {
// host triple instead of the host triple detected by rustc.
.env("RUSTC_WRAPPER", rustc_wrapper)
.env("HOST_TRIPLE", host_triple)
.env("CARGO_CACHE_RUSTC_INFO", "0")
.current_dir(manifest_path.parent().expect("All manifests should have a valid parent."))
.arg("tree")
.arg("--manifest-path")
Expand Down
22 changes: 7 additions & 15 deletions crate_universe/tests/cargo_integration_test.rs
Original file line number Diff line number Diff line change
Expand Up @@ -526,24 +526,16 @@ fn host_specific_build_deps() {
}

let r = runfiles::Runfiles::create().unwrap();

let src_cargo_toml = runfiles::rlocation!(
r,
"rules_rust/crate_universe/test_data/metadata/host_specific_build_deps/Cargo.toml"
)
.unwrap();

// Put Cargo.toml into writable directory structure and create target/ directory to verify that
// cargo does not incorrectly cache rustc info in target/.rustc_info.json file.
let scratch = tempfile::tempdir().unwrap();
let cargo_toml = scratch.path().join("Cargo.toml");
fs::copy(src_cargo_toml, &cargo_toml).unwrap();
fs::create_dir(scratch.path().join("target")).unwrap();

let metadata = run(
"host_specific_build_deps",
HashMap::from([(
cargo_toml.to_string_lossy().to_string(),
runfiles::rlocation!(
r,
"rules_rust/crate_universe/test_data/metadata/host_specific_build_deps/Cargo.toml"
)
.unwrap()
.to_string_lossy()
.to_string(),
"//:test_input".to_string(),
)]),
"rules_rust/crate_universe/test_data/metadata/host_specific_build_deps/Cargo.lock",
Expand Down

0 comments on commit c3f6b84

Please sign in to comment.