From c85a8a1fad20f95bf60ccb95f562d787ca3a42db Mon Sep 17 00:00:00 2001 From: John Ericson Date: Sat, 2 Feb 2019 15:56:52 -0500 Subject: [PATCH 1/2] postgresql: No need to refer to `super` self.callPackage works fine for now. If/when that changes, we have a repo-wide thing to deal with so no reason to do anything now. --- pkgs/servers/sql/postgresql/default.nix | 14 +++++++------- pkgs/top-level/all-packages.nix | 2 +- 2 files changed, 8 insertions(+), 8 deletions(-) diff --git a/pkgs/servers/sql/postgresql/default.nix b/pkgs/servers/sql/postgresql/default.nix index 0624998e5deba5..842d01a640cb8b 100644 --- a/pkgs/servers/sql/postgresql/default.nix +++ b/pkgs/servers/sql/postgresql/default.nix @@ -153,9 +153,9 @@ let ''; }; -in self: super: { +in self: { - postgresql_9_4 = super.callPackage generic { + postgresql_9_4 = self.callPackage generic { version = "9.4.20"; psqlSchema = "9.4"; sha256 = "0zzqjz5jrn624hzh04drpj6axh30a9k6bgawid6rwk45nbfxicgf"; @@ -163,7 +163,7 @@ in self: super: { inherit self; }; - postgresql_9_5 = super.callPackage generic { + postgresql_9_5 = self.callPackage generic { version = "9.5.15"; psqlSchema = "9.5"; sha256 = "0i2lylgmsmy2g1ixlvl112fryp7jmrd0i2brk8sxb7vzzpg3znnv"; @@ -171,7 +171,7 @@ in self: super: { inherit self; }; - postgresql_9_6 = super.callPackage generic { + postgresql_9_6 = self.callPackage generic { version = "9.6.11"; psqlSchema = "9.6"; sha256 = "0c55akrkzqd6p6a8hr0338wk246hl76r9j16p4zn3s51d7f0l99q"; @@ -179,7 +179,7 @@ in self: super: { inherit self; }; - postgresql_10 = super.callPackage generic { + postgresql_10 = self.callPackage generic { version = "10.6"; psqlSchema = "10.0"; sha256 = "0jv26y3f10svrjxzsgqxg956c86b664azyk2wppzpa5x11pjga38"; @@ -187,7 +187,7 @@ in self: super: { inherit self; }; - postgresql_11 = super.callPackage generic { + postgresql_11 = self.callPackage generic { version = "11.1"; psqlSchema = "11.1"; sha256 = "026v0sicsh7avzi45waf8shcbhivyxmi7qgn9fd1x0vl520mx0ch"; @@ -195,4 +195,4 @@ in self: super: { inherit self; }; -} \ No newline at end of file +} diff --git a/pkgs/top-level/all-packages.nix b/pkgs/top-level/all-packages.nix index d81e26abe1deea..6b84b21eb0bb6b 100644 --- a/pkgs/top-level/all-packages.nix +++ b/pkgs/top-level/all-packages.nix @@ -13870,7 +13870,7 @@ in timescaledb-parallel-copy = callPackage ../development/tools/database/timescaledb-parallel-copy { }; - inherit (import ../servers/sql/postgresql pkgs super) + inherit (import ../servers/sql/postgresql pkgs) postgresql_9_4 postgresql_9_5 postgresql_9_6 From 755e824291623e9610fb1a5b8656dab5901e236c Mon Sep 17 00:00:00 2001 From: John Ericson Date: Sat, 2 Feb 2019 15:58:17 -0500 Subject: [PATCH 2/2] all-packages: Just refer to `self`, not `super`, or `res` This ends a years-long process to removoe pointless fixed points in this file! --- pkgs/top-level/all-packages.nix | 20 +++----------------- pkgs/top-level/stage.nix | 5 +---- 2 files changed, 4 insertions(+), 21 deletions(-) diff --git a/pkgs/top-level/all-packages.nix b/pkgs/top-level/all-packages.nix index 6b84b21eb0bb6b..c587082053fa22 100644 --- a/pkgs/top-level/all-packages.nix +++ b/pkgs/top-level/all-packages.nix @@ -6,25 +6,11 @@ * Hint: ### starts category names. */ { lib, noSysDirs, config}: -res: pkgs: super: +pkgs: with pkgs; -let - self = - builtins.trace '' - It seems that you are using a patched Nixpkgs that references the self - variable in pkgs/top-level/all-packages.nix. This variable was incorrectly - named, so its usage needs attention. Please use pkgs for packages or super - for functions. - '' - res; # Do *NOT* use res in your fork. It will be removed. - - # TODO: turn self into an error - -in { - # Allow callPackage to fill in the pkgs argument inherit pkgs; @@ -16218,7 +16204,7 @@ in }; bitwig-studio2 = callPackage ../applications/audio/bitwig-studio/bitwig-studio2.nix { inherit (gnome3) zenity; - inherit (res) bitwig-studio1; + inherit (pkgs) bitwig-studio1; }; bitwig-studio = bitwig-studio2; @@ -22528,7 +22514,7 @@ in parameter set to the right value for your deployment target. */ nixos = configuration: - (import (res.path + "/nixos/lib/eval-config.nix") { + (import (pkgs.path + "/nixos/lib/eval-config.nix") { inherit (pkgs.stdenv.hostPlatform) system; modules = [( { lib, ... }: { diff --git a/pkgs/top-level/stage.nix b/pkgs/top-level/stage.nix index 6ca370e0b9b770..0991c9138bdab9 100644 --- a/pkgs/top-level/stage.nix +++ b/pkgs/top-level/stage.nix @@ -89,10 +89,7 @@ let splice = self: super: import ./splice.nix lib self (buildPackages != null); allPackages = self: super: - let res = import ./all-packages.nix - { inherit lib noSysDirs config; } - res self super; - in res; + import ./all-packages.nix { inherit lib noSysDirs config; } self; aliases = self: super: lib.optionalAttrs (config.allowAliases or true) (import ./aliases.nix lib self super);