Skip to content

Commit

Permalink
Bug 1950375 - Remove exclusion for Windows when handling symlinks whi…
Browse files Browse the repository at this point in the history
…le dealing with PreprocessedFile. r=firefox-build-system-reviewers,sergesanspaille

We used to not use symbolink links on Windows, but that changed a while
back. There was a leftover exclusion causing problems when replacing a
symbolic link with a preprocessed file.

Differential Revision: https://phabricator.services.mozilla.com/D239860
  • Loading branch information
glandium committed Feb 27, 2025
1 parent d6e1111 commit 3166bb7
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions python/mozbuild/mozpack/files.py
Original file line number Diff line number Diff line change
Expand Up @@ -551,10 +551,8 @@ def copy(self, dest, skip_if_older=True):
# destination is not a symlink, we leave it alone, since we're going to
# overwrite its contents anyway.
# If symlinks aren't supported at all, we can skip this step.
# See comment in AbsoluteSymlinkFile about Windows.
if hasattr(os, "symlink") and platform.system() != "Windows":
if os.path.islink(dest.path):
os.remove(dest.path)
if hasattr(os, "symlink") and os.path.islink(dest.path):
os.remove(dest.path)

pp_deps = set(self.extra_depends)

Expand Down

0 comments on commit 3166bb7

Please sign in to comment.