normpath

Author: burrbull

CHANGELOG-rust.md

@@ -6,6 +6,7 @@ This changelog tracks the Rust `svdtools` project. See
 ## [Unreleased]
 
 * cluster add/modify
+* use `normpath` instead of std::fs::canonicalize
 
 ## [v0.2.8] 2022-01-28
 

Cargo.toml

@@ -41,6 +41,7 @@ globset = "0.4.8"
 commands = "0.0.5"
 env_logger = "0.9"
 log = { version = "~0.4", features = ["std"] }
+normpath = "1.1.0"
 
 [dev-dependencies]
 tempfile = "3.3"

src/patch/mod.rs

@@ -15,7 +15,7 @@ use svd_parser::SVDError::DimIndexParse;
 use svd_rs::{DimElement, DimElementBuilder, MaybeArray};
 use yaml_rust::{yaml::Hash, Yaml, YamlLoader};
 
-use anyhow::{anyhow, Context, Ok, Result};
+use anyhow::{anyhow, Context, Result};
 pub type PatchResult = anyhow::Result<()>;
 
 mod device;
@@ -72,7 +72,14 @@ pub fn process_file(yaml_file: &Path) -> Result<()> {
 
 /// Gets the absolute path of relpath from the point of view of frompath.
 fn abspath(frompath: &Path, relpath: &Path) -> Result<PathBuf, std::io::Error> {
-    std::fs::canonicalize(frompath.parent().unwrap().join(relpath))
+    normpath::BasePath::new(frompath)
+        .unwrap()
+        .parent()
+        .unwrap()
+        .unwrap()
+        .join(relpath)
+        .canonicalize()
+        .map(|b| b.as_path().into())
 }
 
 /// Recursively loads any included YAML files.

src/test_utils.rs

@@ -39,5 +39,14 @@ pub fn get_patcher(test_subdir: &Path) -> Result<(Device, Hash)> {
 
 /// Gets the absolute path of relpath from the point of view of frompath.
 fn abspath(frompath: &Path, relpath: &Path) -> PathBuf {
-    std::fs::canonicalize(frompath.parent().unwrap().join(relpath)).unwrap()
+    normpath::BasePath::new(frompath)
+        .unwrap()
+        .parent()
+        .unwrap()
+        .unwrap()
+        .join(relpath)
+        .canonicalize()
+        .unwrap()
+        .as_path()
+        .into()
 }