nixpkgs/pkgs/development/python-modules/jpylyzer/default.nix
adisbladis 02dab4ab5c python3.pkgs.*: Explicitly pass buildPythonPackage format parameter
Long term we should move everything over to `pyproject = true`, but in
the mean time we can work towards deprecating the implicit `format` paremeter.

cc https://github.com/NixOS/nixpkgs/issues/253154
cc @mweinelt @figsoda
2023-12-07 17:46:49 +01:00

65 lines
1.7 KiB
Nix

{ lib
, stdenv
, fetchFromGitHub
, buildPythonPackage
, six
, lxml
, pytestCheckHook
, doFullCheck ? false # weird filenames cause issues on some filesystems
# for passthru.tests
, jpylyzer
}:
let
# unclear relationship between test-files version and jpylyzer version.
# upstream appears to just always test against the latest version, so
# probably worth updating this when package is bumped.
testFiles = fetchFromGitHub {
owner = "openpreserve";
repo = "jpylyzer-test-files";
rev = "146cb0029b5ea9d8ef22dc6683cec8afae1cc63a";
hash = "sha256-uKUau7mYXqGs4dSnXGPnPsH9k81ZCK0aPj5F9HWBMZ8=";
};
in buildPythonPackage rec {
pname = "jpylyzer";
version = "2.1.0";
format = "setuptools";
src = fetchFromGitHub {
owner = "openpreserve";
repo = pname;
rev = version;
hash = "sha256-LBVOwjWC/HEvGgoi8WxEdl33M4JrfdHEj1Dk7f1NAiA=";
};
propagatedBuildInputs = [ six ];
nativeCheckInputs = [ pytestCheckHook lxml ];
# don't depend on testFiles unless doFullCheck as it may not be extractable
# on some filesystems due to weird filenames
preCheck = lib.optionalString doFullCheck ''
sed -i '/^testFilesDir = /ctestFilesDir = "${testFiles}"' tests/unit/test_testfiles.py
'';
disabledTestPaths = lib.optionals (!doFullCheck) [
"tests/unit/test_testfiles.py"
];
pythonImportsCheck = [ "jpylyzer" ];
disallowedReferences = [ testFiles ];
passthru.tests = {
withFullCheck = jpylyzer.override { doFullCheck = true; };
};
meta = with lib; {
description = "JP2 (JPEG 2000 Part 1) image validator and properties extractor";
homepage = "https://jpylyzer.openpreservation.org/";
license = licenses.lgpl3;
maintainers = with maintainers; [ ris ];
};
}