WIP: std::path::absolute to std::fs::canonicalize

paritytech · Sep 2, 2024 · 5856dca · 5856dca
1 parent b74e6ca
commit 5856dca
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/substrate/utils/frame/benchmarking-cli/src/pallet/command.rs b/substrate/utils/frame/benchmarking-cli/src/pallet/command.rs
@@ -750,7 +750,8 @@ impl PalletCmd {
 		state: &'a BenchmarkingState<H>,
 	) -> Result<FetchedCode<'a, BenchmarkingState<H>, H>> {
 		if let Some(runtime) = &self.runtime {
-			let runtime = std::path::absolute(runtime)
+			// let runtime = std::path::absolute(runtime) - FAIL-CI
+			let runtime = std::fs::canonicalize(runtime)
 				.map_err(|e| format!("Could not get absolute path for runtime file: {e}"))?;
 			log::info!("Loading WASM from {}", runtime.display());
 

diff --git a/substrate/utils/frame/benchmarking-cli/src/pallet/writer.rs b/substrate/utils/frame/benchmarking-cli/src/pallet/writer.rs
@@ -484,7 +484,8 @@ pub(crate) fn write_results(
 			benchmarks: results.clone(),
 		};
 
-		let file_path = std::path::absolute(&file_path).map_err(|e| {
+		// let file_path = std::path::absolute(&file_path).map_err(|e| {  - FAIL-CI
+		let file_path = std::fs::canonicalize(&file_path).map_err(|e| {
 			format!("Could not get absolute path for: {:?}. Error: {:?}", &file_path, e)
 		})?;
 		let mut output_file = fs::File::create(&file_path).map_err(|e| {