Avoid macos in firefox activations

kachick · Jun 28, 2024 · d80860b · d80860b
1 parent fcca15a
commit d80860b
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 3 deletions.
diff --git a/home-manager/common.nix b/home-manager/common.nix
@@ -16,7 +16,7 @@
     ./ssh.nix
     ./git.nix
     ./editor.nix
-    ./browser.nix
+    ./linux.nix
     ./darwin.nix
   ];
 

diff --git a/home-manager/browser.nix → home-manager/firefox.nix b/home-manager/browser.nix → home-manager/firefox.nix
@@ -2,6 +2,7 @@
 
 {
   # https://github.com/nix-community/home-manager/blob/release-24.05/modules/programs/firefox.nix
+  # Firefox package does not support both M1 and Intel like x86_64-apple-darwin
   programs.firefox = {
     enable = true;
     profiles.default = {
@@ -25,6 +26,4 @@
       userChrome = builtins.readFile ../config/Firefox/userChrome.css;
     };
   };
-
-  # Don't add unfree packages like google-chrome here for using in containers
 }
diff --git a/home-manager/linux.nix b/home-manager/linux.nix
@@ -0,0 +1,4 @@
+{ pkgs, lib, ... }:
+
+# https://github.com/nix-community/home-manager/issues/414#issuecomment-427163925
+lib.mkMerge [ (lib.mkIf pkgs.stdenv.isLinux { imports = [ ./firefox.nix ]; }) ]