Skip to content

Commit

Permalink
python312Packages: cleanup instances of doCheck = true
Browse files Browse the repository at this point in the history
This is implicitly the case, and unless `doCheck` is referenced it makes
no sense setting it.
  • Loading branch information
mweinelt committed Oct 25, 2024
1 parent 6dbbd5a commit 266c161
Show file tree
Hide file tree
Showing 45 changed files with 1 addition and 66 deletions.
1 change: 0 additions & 1 deletion pkgs/development/python-modules/anitopy/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,6 @@ buildPythonPackage rec {
};

pythonImportsCheck = [ "anitopy" ];
doCheck = true;

meta = with lib; {
description = "Python library for parsing anime video filenames";
Expand Down
1 change: 0 additions & 1 deletion pkgs/development/python-modules/beziers/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,6 @@ buildPythonPackage rec {

propagatedBuildInputs = [ pyclipper ];

doCheck = true;
nativeCheckInputs = [
dotmap
matplotlib
Expand Down
1 change: 0 additions & 1 deletion pkgs/development/python-modules/collidoscope/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,6 @@ buildPythonPackage rec {
uharfbuzz
];

doCheck = true;
nativeCheckInputs = [ unittestCheckHook ];
unittestFlagsArray = [
"-s"
Expand Down
1 change: 0 additions & 1 deletion pkgs/development/python-modules/commandlines/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,6 @@ buildPythonPackage rec {
hash = "sha256-x3iUeOTAaTKNW5Y5foMPMJcWVxu52uYZoY3Hhe3UvQ4=";
};

doCheck = true;
nativeCheckInputs = [ pytestCheckHook ];

meta = with lib; {
Expand Down
2 changes: 0 additions & 2 deletions pkgs/development/python-modules/commitizen/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -77,8 +77,6 @@ buildPythonPackage rec {
pytest7CheckHook
];

doCheck = true;

pythonImportsCheck = [ "commitizen" ];

# The tests require a functional git installation
Expand Down
1 change: 0 additions & 1 deletion pkgs/development/python-modules/dehinter/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,6 @@ buildPythonPackage rec {

propagatedBuildInputs = [ fonttools ];

doCheck = true;
nativeCheckInputs = [ pytestCheckHook ];

meta = with lib; {
Expand Down
3 changes: 0 additions & 3 deletions pkgs/development/python-modules/dissect-jffs/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -33,9 +33,6 @@ buildPythonPackage rec {
dissect-util
];

# Test file handling fails
doCheck = true;

pythonImportsCheck = [ "dissect.jffs" ];

meta = with lib; {
Expand Down
2 changes: 0 additions & 2 deletions pkgs/development/python-modules/dropbox/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -52,8 +52,6 @@ buildPythonPackage rec {
--replace "'pytest-runner==5.2.0'," ""
'';

doCheck = true;

pythonImportsCheck = [ "dropbox" ];

nativeBuildInputs = [
Expand Down
1 change: 0 additions & 1 deletion pkgs/development/python-modules/eigenpy/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,6 @@ buildPythonPackage rec {
numpy
];

doCheck = true;
pythonImportsCheck = [ "eigenpy" ];

meta = with lib; {
Expand Down
1 change: 0 additions & 1 deletion pkgs/development/python-modules/font-v/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,6 @@ buildPythonPackage rec {
gitpython
];

doCheck = true;
nativeCheckInputs = [
git
pytestCheckHook
Expand Down
1 change: 0 additions & 1 deletion pkgs/development/python-modules/fontfeatures/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,6 @@ buildPythonPackage rec {
youseedee
];

doCheck = true;
nativeCheckInputs = [ pytestCheckHook ];
disabledTestPaths = [
# These tests require babelfont but we have to leave it out and skip them
Expand Down
2 changes: 0 additions & 2 deletions pkgs/development/python-modules/gst-python/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -53,8 +53,6 @@ buildPythonPackage rec {
"-Dpython=${python.pythonOnBuildForHost.interpreter}"
];

doCheck = true;

# TODO: Meson setup hook does not like buildPythonPackage
# https://github.com/NixOS/nixpkgs/issues/47390
installCheckPhase = "meson test --print-errorlogs";
Expand Down
1 change: 0 additions & 1 deletion pkgs/development/python-modules/iwlib/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,6 @@ buildPythonPackage rec {
nativeBuildInputs = [ pytest ];
pythonImportsCheck = [ "iwlib" ];

doCheck = true;
checkInputs = [ pytest ];
checkPhase = "python iwlib/_iwlib_build.py; pytest -v";

Expand Down
1 change: 0 additions & 1 deletion pkgs/development/python-modules/kinparse/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,6 @@ buildPythonPackage {
sha256 = "1nrjnybwzy93c79yylcwmb4lvkx7hixavnjwffslz0zwn32l0kx3";
};

doCheck = true;
pythonImportsCheck = [ "kinparse" ];

nativeCheckInputs = [ pytest ];
Expand Down
1 change: 0 additions & 1 deletion pkgs/development/python-modules/kurbopy/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,6 @@ buildPythonPackage rec {
hash = "sha256-W0BebCXC1wqwtQP+zHjISxSJjXHD9U6p9eNS12Nfb2Y=";
};

doCheck = true;
nativeCheckInputs = [ pytestCheckHook ];
preCheck = ''
# pytestCheckHook puts . at the front of Python's sys.path, due to:
Expand Down
1 change: 0 additions & 1 deletion pkgs/development/python-modules/lcov-cobertura/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,6 @@ buildPythonPackage rec {
build-system = [ setuptools ];
dependencies = [ distutils ];

doCheck = true;
pythonImportsCheck = [ "lcov_cobertura" ];

meta = {
Expand Down
1 change: 0 additions & 1 deletion pkgs/development/python-modules/meep/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -122,7 +122,6 @@ buildPythonPackage rec {
(calls `sim.run()`), as only then MPI will be initialised and MPI linking
errors can be caught.
*/
doCheck = true;
nativeCheckInputs = [
mpiCheckPhaseHook
openssh
Expand Down
1 change: 0 additions & 1 deletion pkgs/development/python-modules/opentypespec/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,6 @@ buildPythonPackage rec {
hash = "sha256-fOEHmtlCkFhn1jyIA+CsHIfud7x3PPb7UWQsnrVyDqY=";
};

doCheck = true;
nativeCheckInputs = [ unittestCheckHook ];
unittestFlagsArray = [
"-s"
Expand Down
1 change: 0 additions & 1 deletion pkgs/development/python-modules/ots-python/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,6 @@ buildPythonPackage rec {
propagatedBuildInputs = [ opentype-sanitizer ];
nativeBuildInputs = [ setuptools-scm ];

doCheck = true;
nativeCheckInputs = [ pytestCheckHook ];

meta = with lib; {
Expand Down
2 changes: 0 additions & 2 deletions pkgs/development/python-modules/pid/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -29,8 +29,6 @@ buildPythonPackage rec {

nativeCheckInputs = [ pytestCheckHook ];

doCheck = true;

meta = with lib; {
description = "Pidfile featuring stale detection and file-locking";
homepage = "https://github.com/trbs/pid/";
Expand Down
1 change: 0 additions & 1 deletion pkgs/development/python-modules/pyarrow/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,6 @@ buildPythonPackage rec {
PARQUET_HOME = arrow-cpp;

ARROW_TEST_DATA = lib.optionalString doCheck arrow-cpp.ARROW_TEST_DATA;

doCheck = true;

dontUseCmakeConfigure = true;
Expand Down
2 changes: 0 additions & 2 deletions pkgs/development/python-modules/pycangjie/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -43,8 +43,6 @@ buildPythonPackage {

configureScript = "./autogen.sh";

doCheck = true;

meta = with lib; {
description = "Python wrapper to libcangjie";
homepage = "http://cangjians.github.io/projects/pycangjie/";
Expand Down
1 change: 0 additions & 1 deletion pkgs/development/python-modules/pymoo/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,6 @@ buildPythonPackage rec {
scipy
];

doCheck = true;
preCheck = ''
substituteInPlace pymoo/config.py \
--replace-fail "https://raw.githubusercontent.com/anyoptimization/pymoo-data/main/" \
Expand Down
1 change: 0 additions & 1 deletion pkgs/development/python-modules/pyqt/6.x.nix
Original file line number Diff line number Diff line change
Expand Up @@ -130,7 +130,6 @@ buildPythonPackage rec {
dontConfigure = true;

# Checked using pythonImportsCheck, has no tests
doCheck = true;

pythonImportsCheck =
[
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,6 @@ buildPythonPackage rec {
dontConfigure = true;

# Checked using pythonImportsCheck, has no tests
doCheck = true;

pythonImportsCheck = [
"PyQt6.QtWebEngineCore"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -60,8 +60,6 @@ buildPythonPackage rec {
requests
];

doCheck = true;

nativeCheckInputs = [
requests-mock
stestr
Expand Down
2 changes: 0 additions & 2 deletions pkgs/development/python-modules/python-musicpd/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -19,8 +19,6 @@ buildPythonPackage rec {

build-system = [ setuptools ];

doCheck = true;

meta = with lib; {
description = "MPD (Music Player Daemon) client library written in pure Python";
homepage = "https://gitlab.com/kaliko/python-musicpd";
Expand Down
5 changes: 1 addition & 4 deletions pkgs/development/python-modules/pyytlounge/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@
buildPythonPackage rec {
pname = "pyytlounge";
version = "2.1.1";
pyproject = true;

src = fetchFromGitHub {
owner = "FabioGNR";
Expand All @@ -21,10 +22,6 @@ buildPythonPackage rec {
hash = "sha256-0QPa3EzOBv5fuw3FGgmoN4KiC4KHo1Z+Svjcneoe0pc=";
};

pyproject = true;

doCheck = true;

build-system = [ hatchling ];

dependencies = [ aiohttp ];
Expand Down
2 changes: 0 additions & 2 deletions pkgs/development/python-modules/rasterio/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -88,8 +88,6 @@ buildPythonPackage rec {
shapely
];

doCheck = true;

preCheck = ''
rm -r rasterio # prevent importing local rasterio
'';
Expand Down
2 changes: 0 additions & 2 deletions pkgs/development/python-modules/rouge-score/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -52,8 +52,6 @@ buildPythonPackage rec {

nativeCheckInputs = [ pytestCheckHook ];

doCheck = true;

disabledTests = [
# https://github.com/google-research/google-research/issues/1203
"testRougeLSumSentenceSplitting"
Expand Down
1 change: 0 additions & 1 deletion pkgs/development/python-modules/rstr/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,6 @@ buildPythonPackage rec {
setuptools-scm
];

doCheck = true;
nativeCheckInputs = [ unittestCheckHook ];

meta = with lib; {
Expand Down
1 change: 0 additions & 1 deletion pkgs/development/python-modules/scikits-odes/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,6 @@ buildPythonPackage rec {
scipy
] ++ lib.optionals (!isPy3k) [ enum34 ];

doCheck = true;
nativeCheckInputs = [ pytest ];

checkPhase = ''
Expand Down
1 change: 0 additions & 1 deletion pkgs/development/python-modules/segyio/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,6 @@ buildPythonPackage rec {
scikit-build
];

doCheck = true;
# I'm not modifying the checkPhase nor adding a pytestCheckHook because the pytest is called
# within the cmake test phase
nativeCheckInputs = [
Expand Down
2 changes: 0 additions & 2 deletions pkgs/development/python-modules/sip/4.x.nix
Original file line number Diff line number Diff line change
Expand Up @@ -57,8 +57,6 @@ buildPythonPackage rec {
"sipconfig"
];

doCheck = true;

meta = with lib; {
description = "Creates C++ bindings for Python modules";
mainProgram = "sip";
Expand Down
2 changes: 0 additions & 2 deletions pkgs/development/python-modules/skyfield/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -49,8 +49,6 @@ buildPythonPackage rec {
assay
];

doCheck = true;

checkPhase = ''
runHook preCheck
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,6 @@ stdenv.mkDerivation {

dontConfigure = true;
dontBuild = true;
doCheck = true;

nativeCheckInputs = [
pytest
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,6 @@ stdenv.mkDerivation {

dontConfigure = true;
dontBuild = true;
doCheck = true;

nativeCheckInputs = [
pytest
Expand Down
1 change: 0 additions & 1 deletion pkgs/development/python-modules/sre-yield/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,6 @@ buildPythonPackage rec {

nativeBuildInputs = [ setuptools ];

doCheck = true;
nativeCheckInputs = [ unittestCheckHook ];

meta = with lib; {
Expand Down
2 changes: 0 additions & 2 deletions pkgs/development/python-modules/telegraph/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -34,8 +34,6 @@ buildPythonPackage rec {

disabledTests = [ "test_get_page" ];

doCheck = true;

pythonImportsCheck = [ "telegraph" ];

meta = with lib; {
Expand Down
2 changes: 0 additions & 2 deletions pkgs/development/python-modules/tpm2-pytss/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -81,8 +81,6 @@ buildPythonPackage rec {
pyyaml
];

doCheck = true;

nativeCheckInputs = [
pytestCheckHook
tpm2-tools
Expand Down
2 changes: 0 additions & 2 deletions pkgs/development/python-modules/turnt/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -24,8 +24,6 @@ buildPythonPackage rec {
tomli
];

doCheck = true;

checkPhase = ''
runHook preCheck
$out/bin/turnt test/*/*.t
Expand Down
1 change: 0 additions & 1 deletion pkgs/development/python-modules/ufolint/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,6 @@ buildPythonPackage rec {
fonttools
];

doCheck = true;
nativeBuildInputs = [ pytestCheckHook ];

meta = with lib; {
Expand Down
2 changes: 0 additions & 2 deletions pkgs/development/python-modules/xapian/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -34,8 +34,6 @@ buildPythonPackage rec {
xapian
];

doCheck = true;

checkPhase = ''
${python.interpreter} python${pythonSuffix}/pythontest.py
'';
Expand Down
2 changes: 0 additions & 2 deletions pkgs/development/python-modules/xdot/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -60,8 +60,6 @@ buildPythonPackage rec {
runHook postCheck
'';

doCheck = true;

meta = with lib; {
description = "Interactive viewer for graphs written in Graphviz's dot";
mainProgram = "xdot";
Expand Down
1 change: 0 additions & 1 deletion pkgs/development/python-modules/youseedee/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,6 @@ buildPythonPackage rec {
requests
];

doCheck = true;
# Package has no unit tests, but we can check an example as per README.rst:
checkPhase = ''
runHook preCheck
Expand Down

0 comments on commit 266c161

Please sign in to comment.