From cab045f4f50a3e80eedbdf8953e10cd53f029b43 Mon Sep 17 00:00:00 2001 From: aszlig Date: Fri, 25 Aug 2017 09:56:49 +0200 Subject: devhell/profiles: Fix package overrides Regression introduced by 5990a4d62f2d3146ddd03ec2461d45b2102e5f42. In this commit, I copied over the nixpkgs.config.packageOverrides attribute, but didn't trim off the "nixpkgs.config", so the packageOverrides attribute didn't get evaluated correctly. While converting everything to overlays in 0feae9218e6f922366665e6515bc, I noticed that for @devhell's profile the store paths were different. In hindsight, this is pretty obvious given that overrides didn't apply *at* *all* but with the move to nixpkgs.overlays they suddenly applied. So this moves the overrides to overlays instead and drops the override on gnupg, because gnupg is already gnupg21 in upstream . Signed-off-by: aszlig Cc: @devhell --- modules/user/devhell/profiles/packages.nix | 24 +++++++++++------------- 1 file changed, 11 insertions(+), 13 deletions(-) diff --git a/modules/user/devhell/profiles/packages.nix b/modules/user/devhell/profiles/packages.nix index faaa5bfd..c8f61ad9 100644 --- a/modules/user/devhell/profiles/packages.nix +++ b/modules/user/devhell/profiles/packages.nix @@ -9,6 +9,17 @@ in { }; config = lib.mkIf cfg.enable { + nixpkgs.overlays = lib.singleton (lib.const (super: { + ncmpcpp = super.ncmpcpp.override { + visualizerSupport = true; + clockSupport = true; + }; + + sox = super.sox.override { + enableLame = true; + }; + })); + nixpkgs.config = { pulseaudio = true; @@ -37,19 +48,6 @@ in { mpv = { youtubeSupport = true; }; - - nixpkgs.config.packageOverrides = super: { - gnupg = super.gnupg21; - - ncmpcpp = super.ncmpcpp.override { - visualizerSupport = true; - clockSupport = true; - }; - - sox = super.sox.override { - enableLame = true; - }; - }; }; environment.systemPackages = with pkgs; [ -- cgit 1.4.1