From e4a92b858d534e234e79bbbba0f707c41e6e9c1e Mon Sep 17 00:00:00 2001 From: Wolfgang Walther Date: Mon, 20 Oct 2025 15:05:51 +0200 Subject: [PATCH] plfit: python -> withPython Improves the override interface, because `withPython` does not conflict with the old `python` package. --- pkgs/by-name/pl/plfit/package.nix | 9 ++++++--- pkgs/top-level/all-packages.nix | 4 ---- pkgs/top-level/python-packages.nix | 7 ++++++- 3 files changed, 12 insertions(+), 8 deletions(-) diff --git a/pkgs/by-name/pl/plfit/package.nix b/pkgs/by-name/pl/plfit/package.nix index 90f103aea0e3..c0fd37234e2d 100644 --- a/pkgs/by-name/pl/plfit/package.nix +++ b/pkgs/by-name/pl/plfit/package.nix @@ -3,11 +3,14 @@ fetchFromGitHub, lib, llvmPackages, + withPython ? false, python ? null, stdenv, swig, }: +assert withPython -> python != null; + stdenv.mkDerivation (finalAttrs: { pname = "plfit"; version = "1.0.1"; @@ -19,7 +22,7 @@ stdenv.mkDerivation (finalAttrs: { hash = "sha256-0JrPAq/4yzr7XbxvcnFj8CKmMyZT05PkSdGprNdAsJA="; }; - postPatch = lib.optionalString (python != null) '' + postPatch = lib.optionalString withPython '' substituteInPlace src/CMakeLists.txt \ --replace-fail ' ''${Python3_SITEARCH}' ' ${placeholder "out"}/${python.sitePackages}' \ --replace-fail ' ''${Python3_SITELIB}' ' ${placeholder "out"}/${python.sitePackages}' @@ -28,7 +31,7 @@ stdenv.mkDerivation (finalAttrs: { nativeBuildInputs = [ cmake ] - ++ lib.optionals (python != null) [ + ++ lib.optionals withPython [ python swig ]; @@ -36,7 +39,7 @@ stdenv.mkDerivation (finalAttrs: { cmakeFlags = [ "-DPLFIT_USE_OPENMP=ON" ] - ++ lib.optionals (python != null) [ + ++ lib.optionals withPython [ "-DPLFIT_COMPILE_PYTHON_MODULE=ON" ]; diff --git a/pkgs/top-level/all-packages.nix b/pkgs/top-level/all-packages.nix index 01921036db84..82dd191ad20e 100644 --- a/pkgs/top-level/all-packages.nix +++ b/pkgs/top-level/all-packages.nix @@ -3453,10 +3453,6 @@ with pkgs; tautulli = python3Packages.callPackage ../servers/tautulli { }; - plfit = callPackage ../by-name/pl/plfit/package.nix { - python = null; - }; - inherit (callPackage ../development/tools/pnpm { }) pnpm_8 pnpm_9 diff --git a/pkgs/top-level/python-packages.nix b/pkgs/top-level/python-packages.nix index da84d3a7976f..7f82c423d97e 100644 --- a/pkgs/top-level/python-packages.nix +++ b/pkgs/top-level/python-packages.nix @@ -12340,7 +12340,12 @@ self: super: with self; { plexwebsocket = callPackage ../development/python-modules/plexwebsocket { }; - plfit = toPythonModule (pkgs.plfit.override { inherit (self) python; }); + plfit = toPythonModule ( + pkgs.plfit.override { + withPython = true; + inherit (self) python; + } + ); plink = callPackage ../development/python-modules/plink { };