Merge pull request #327220 from pyrox0/denose/cmake-lint

cmake-lint: remove nose dependency
NixOS · Jul 24, 2024 · ae9c93b · ae9c93b
2 parents f395d44 + 70c7038
commit ae9c93b
Showing 1 changed file with 9 additions and 8 deletions.
diff --git a/pkgs/by-name/cm/cmake-lint/package.nix b/pkgs/by-name/cm/cmake-lint/package.nix
@@ -18,19 +18,20 @@ python3Packages.buildPythonApplication rec {
     hash = "sha256-/OuWwerBlJynEibaYo+jkLpHt4x9GZrqMRJNxgrDBlM=";
   };
 
-  nativeBuildInputs = [ python3Packages.setuptools ];
+  postPatch = ''
+    # We don't need to test coverage, so remove these checks
+    substituteInPlace setup.cfg \
+      --replace-fail "addopts = --cov-fail-under=84 --cov=./cmakelint" ""
+  '';
+
+  build-system = [ python3Packages.setuptools ];
 
   pythonImportsCheck = [ "cmakelint" ];
 
-  nativeCheckInputs = with python3Packages; [
-    pytestCheckHook
-    nose
+  nativeCheckInputs = [
+    python3Packages.pytestCheckHook
   ];
 
-  checkPhase = ''
-    nosetests
-  '';
-
   passthru.tests = {
     version = testers.testVersion { package = cmake-lint; };
   };