Closed miraclx closed 1 year ago
Apparently, the paths returned from cargo_metadata like Metadata::{target_directory,workspace_root} aren't absolute.
cargo_metadata
Metadata::{
target_directory
,
workspace_root
}
This assumed guarantee causes the logic in util::copy to be invalidated.
util::copy
Treating the same file as if it were different, because one of the paths had ../.. in it.
../..
This patch migrates to early canonization, to avoid this error coming up again.
Props to @itegulov, for discovering this. We should probably cover this in tests.
Once merged, #76 fixes the CI issue here.
Apparently, the paths returned from
cargo_metadata
likeMetadata::{
target_directory
,
workspace_root
}
aren't absolute.This assumed guarantee causes the logic in
util::copy
to be invalidated.Treating the same file as if it were different, because one of the paths had
../..
in it.This patch migrates to early canonization, to avoid this error coming up again.
Props to @itegulov, for discovering this. We should probably cover this in tests.