From 56b2812a853bf522aaa2ba5212cf1a7ba39b1710 Mon Sep 17 00:00:00 2001 From: Ed Page Date: Wed, 14 Jun 2023 12:25:59 -0500 Subject: [PATCH] fix(embedded): Don't append hash to bin names Background: the hash existed for sharing a target directory. That code isn't implemented yet and a per-user build cache might remove the need for it, so let's remove it for now and more carefully weigh adding it back in. Immediate: This reduces the chance of hitting file length issues on Windows. Generally: This is a bit hacky and for an official solution, we should probably try to find a better way. This could become more important as single-file packages are allowed in workspaces. --- src/cargo/util/toml/embedded.rs | 7 +++---- tests/testsuite/script.rs | 36 ++++++++++++++------------------- 2 files changed, 18 insertions(+), 25 deletions(-) diff --git a/src/cargo/util/toml/embedded.rs b/src/cargo/util/toml/embedded.rs index 435d980cb61..c911161dd60 100644 --- a/src/cargo/util/toml/embedded.rs +++ b/src/cargo/util/toml/embedded.rs @@ -142,8 +142,7 @@ fn expand_manifest_(script: &RawScript, config: &Config) -> CargoResult