From 3427e2c8cf36d7ba109da938fb872aad6e3759af Mon Sep 17 00:00:00 2001 From: Harmen Stoppels Date: Thu, 17 Mar 2022 15:18:28 +0100 Subject: PackageViewMixin: fix symlinks conflict issue (#29515) `stat`'ing a file in the dst dir is the wrong thing to do, you should `lstat` to capture broken symlinks. --- lib/spack/spack/package.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/spack/spack/package.py b/lib/spack/spack/package.py index a972350f98..453623d97a 100644 --- a/lib/spack/spack/package.py +++ b/lib/spack/spack/package.py @@ -450,7 +450,7 @@ class PackageViewMixin(object): Alternative implementations may allow some of the files to exist in the view (in this case they would be omitted from the results). """ - return set(dst for dst in merge_map.values() if os.path.exists(dst)) + return set(dst for dst in merge_map.values() if os.path.lexists(dst)) def add_files_to_view(self, view, merge_map): """Given a map of package files to destination paths in the view, add @@ -459,7 +459,7 @@ class PackageViewMixin(object): linked into the view already include the file. """ for src, dst in merge_map.items(): - if not os.path.exists(dst): + if not os.path.lexists(dst): view.link(src, dst, spec=self.spec) def remove_files_from_view(self, view, merge_map): -- cgit v1.2.3-70-g09d2