Skip to content

Commit 700d618

Browse files
committedMay 4, 2017
wrapFirefox: fix #25505 gtk3 file dialogs
Crash + icons.
1 parent 6d3cc8c commit 700d618

File tree

2 files changed

+11
-3
lines changed

2 files changed

+11
-3
lines changed
 

‎pkgs/applications/networking/browsers/firefox/common.nix

+1-1
Original file line numberDiff line numberDiff line change
@@ -200,6 +200,6 @@ stdenv.mkDerivation (rec {
200200
gtk = gtk2;
201201
inherit nspr;
202202
inherit ffmpegSupport;
203-
};
203+
} // lib.optionalAttrs gtk3Support { inherit gtk3; };
204204

205205
} // overrides)

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

+10-2
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,9 @@ in stdenv.mkDerivation {
7474
];
7575
};
7676

77-
buildInputs = [makeWrapper] ++ lib.optionals (!ffmpegSupport) gst-plugins;
77+
buildInputs = [makeWrapper]
78+
++ lib.optional (!ffmpegSupport) gst-plugins
79+
++ lib.optional (browser ? gtk3) browser.gtk3;
7880

7981
buildCommand = ''
8082
if [ ! -x "${browser}/bin/${browserName}" ]
@@ -92,7 +94,13 @@ in stdenv.mkDerivation {
9294
--prefix-contents PATH ':' "$(filterExisting $(addSuffix /extra-bin-path $plugins))" \
9395
--suffix PATH ':' "$out/bin" \
9496
--set MOZ_APP_LAUNCHER "${browserName}${nameSuffix}" \
95-
${lib.optionalString (!ffmpegSupport) ''--prefix GST_PLUGIN_SYSTEM_PATH : "$GST_PLUGIN_SYSTEM_PATH"''}
97+
${lib.optionalString (!ffmpegSupport)
98+
''--prefix GST_PLUGIN_SYSTEM_PATH : "$GST_PLUGIN_SYSTEM_PATH"''
99+
+ lib.optionalString (browser ? gtk3)
100+
''--prefix XDG_DATA_DIRS : "$GSETTINGS_SCHEMAS_PATH" \
101+
--suffix XDG_DATA_DIRS : '${gnome3.defaultIconTheme}/share'
102+
''
103+
}
96104
97105
if [ -e "${browser}/share/icons" ]; then
98106
mkdir -p "$out/share"

1 commit comments

Comments
 (1)

ttuegel commented on May 10, 2017

@ttuegel
Member

@vcunat This fix also needs to be applied to firefox-bin.

Please sign in to comment.