From bbf1c769252adcd3d0411b670f7da86d5d508a3c Mon Sep 17 00:00:00 2001
From: Thiago Kenji Okada <thiagokokada@gmail.com>
Date: Thu, 23 Oct 2025 11:01:03 +0100
Subject: [PATCH] firefox: do not define gtk_modules in darwin

This is causing an unnecessary dependency in libcanberra-gtk3 being
added to the wrapper.

Right now gtk3 derivation is broken so firefox is broken in
aarch64-darwin, but even if gtk3 was working having an unnecessary
dependency being added is a waste of storage.

This commit fixes it.
---
 pkgs/applications/networking/browsers/firefox/wrapper.nix | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pkgs/applications/networking/browsers/firefox/wrapper.nix b/pkgs/applications/networking/browsers/firefox/wrapper.nix
index be43ccc85ad6..c144a5076e75 100644
--- a/pkgs/applications/networking/browsers/firefox/wrapper.nix
+++ b/pkgs/applications/networking/browsers/firefox/wrapper.nix
@@ -116,7 +116,7 @@ let
         ++ lib.optional smartcardSupport opensc
         ++ pkcs11Modules
         ++ lib.optionals (!isDarwin) gtk_modules;
-      gtk_modules = [ libcanberra-gtk3 ];
+      gtk_modules = lib.optionals (!isDarwin) [ libcanberra-gtk3 ];
 
       # Darwin does not rename bundled binaries
       launcherName = "${applicationName}${lib.optionalString (!isDarwin) nameSuffix}";