From 2c8c689cd7643f326e2782d8d8b28572d946816a Mon Sep 17 00:00:00 2001 From: networkException Date: Thu, 15 Aug 2024 02:16:25 +0200 Subject: [PATCH] Revert "nixos/chromium: Make programs.chromium.enable install chromium" This reverts commit 93b6400ff55782bdcdc68df89168fe498ae6494a. Putting chromium in the system closure by enabling the module breaks previous expectations of module users. Previously, this would create policy files for chromium, google-chrome and brave as chromium based browsers. Use cases relating to a browser other than chromium directly as well as only using home-manager's module system to configure other aspects of a chromium package (various use cases require overriding inputs to the chromium derivation) were not covered by this breaking change. More design is needed before having policy and package options coexist properly in this module. --- nixos/modules/programs/chromium.nix | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/nixos/modules/programs/chromium.nix b/nixos/modules/programs/chromium.nix index 298a2197effa..4d248dbe0945 100644 --- a/nixos/modules/programs/chromium.nix +++ b/nixos/modules/programs/chromium.nix @@ -19,8 +19,6 @@ in programs.chromium = { enable = lib.mkEnableOption "{command}`chromium` policies"; - package = lib.mkPackageOption pkgs "chromium" { }; - enablePlasmaBrowserIntegration = lib.mkEnableOption "Native Messaging Host for Plasma Browser Integration"; plasmaBrowserIntegrationPackage = lib.mkPackageOption pkgs [ "plasma5Packages" "plasma-browser-integration" ] { }; @@ -121,9 +119,8 @@ in ###### implementation - config = lib.mkIf cfg.enable { - environment.systemPackages = [ cfg.package ]; - environment.etc = { + config = { + environment.etc = lib.mkIf cfg.enable { # for chromium "chromium/native-messaging-hosts/org.kde.plasma.browser_integration.json" = lib.mkIf cfg.enablePlasmaBrowserIntegration { source = "${cfg.plasmaBrowserIntegrationPackage}/etc/chromium/native-messaging-hosts/org.kde.plasma.browser_integration.json"; };