Skip to content

Commit a2e8864

Browse files
committed
firefox: copy *.dylibs on darwin
Without these, some functionality seems broken (e.g. Crypto API). Resolves #394029 Signed-off-by: Ihar Hrachyshka <[email protected]>
1 parent c0b2096 commit a2e8864

File tree

1 file changed

+4
-2
lines changed
  • pkgs/applications/networking/browsers/firefox

1 file changed

+4
-2
lines changed

pkgs/applications/networking/browsers/firefox/wrapper.nix

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -405,10 +405,12 @@ let
405405
+ lib.optionalString isDarwin ''
406406
cd "${appPath}"
407407
408-
# These files have to be copied and not symlinked, otherwise tabs crash.
408+
# The omni.ja files have to be copied and not symlinked, otherwise tabs crash.
409409
# Maybe related to how omni.ja file is mmapped into memory. See:
410410
# https://github.com/mozilla/gecko-dev/blob/b1662b447f306e6554647914090d4b73ac8e1664/modules/libjar/nsZipArchive.cpp#L204
411-
for file in $(find . -type l -name "omni.ja"); do
411+
#
412+
# The *.dylib files are copied, otherwise some basic functionality, e.g. Crypto API, is broken.
413+
for file in $(find . -name "omni.ja" -o -name "*.dylib"); do
412414
rm "$file"
413415
cp "${browser}/${appPath}/$file" "$file"
414416
done

0 commit comments

Comments
 (0)