fix passing qt5 version to pythonInterpreters

jorsn · jorsn · commit d63977883dea · 2020-09-18T01:04:20.000+02:00
fixes c88f3ad, which resulted in qt 5.15 being used in pythonPackages, despite 5.14 being declared, and adapts qutebrowser accordingly. 'callPackage { pkgs = pkgs // { … }; }' does not work, because it does not take into account the recursive evaluation of nixpkgs: `pkgs/development/interpreters/python/default.nix` calls `pkgs/top-level/python-packages.nix` with `callPackage`. Thus, even if the former gets passed the updated `pkgs`, the latter always gets passed `pkgs.pkgs`. For the change in the qt5 version to apply consistently, 'pkgs.extend' must be used. qutebrowser only used the right qt5 version (5.15) because all pythonPackages used it anyway.
diff --git a/pkgs/development/interpreters/python/default.nix b/pkgs/development/interpreters/python/default.nix
@@ -1,4 +1,4 @@
-{ pkgs, lib }:
+{ pkgs }:
 
 with pkgs;
 
diff --git a/pkgs/top-level/all-packages.nix b/pkgs/top-level/all-packages.nix
@@ -10160,10 +10160,10 @@ in
 
   pythonInterpreters = callPackage ./../development/interpreters/python {
     # Overrides that apply to all Python interpreters
-    pkgs = pkgs // {
+    pkgs = pkgs.extend (pkgs: _: {
       qt5 = pkgs.qt514;
       libsForQt5 = pkgs.libsForQt514;
-    };
+    });
   };
   inherit (pythonInterpreters) python27 python36 python37 python38 python39 python3Minimal pypy27 pypy36;
 
@@ -22836,18 +22836,10 @@ in
   quodlibet-xine-full = quodlibet-full.override { xineBackend = true; tag = "-xine-full"; };
 
   qutebrowser = let
-    libsForQt5 = libsForQt515;
-    qt5 = qt515;
-    python = python3.override {
-      packageOverrides = self: super: {
-        pkgs = pkgs // {
-          inherit libsForQt5 qt5;
-        };
-      };
-      self = python3;
-    };
+    pythonI = pkgs.pythonInterpreters.override { inherit pkgs; };
+    verString = lib.concatStrings (lib.take 2 (lib.splitVersion pkgs.python3.version));
   in libsForQt5.callPackage ../applications/networking/browsers/qutebrowser {
-    python3 = python;
+    python3 = pythonI."python${verString}";
   };
 
   rabbitvcs = callPackage ../applications/version-management/rabbitvcs {};

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-{ pkgs, lib }:`
	`1`	`+{ pkgs }:`
`2`	`2`
`3`	`3`	`with pkgs;`
`4`	`4`