New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
tumpa: init at 0.1.1 #108522
tumpa: init at 0.1.1 #108522
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you take a look at https://github.com/NixOS/nixpkgs/pull/108378/files#diff-a6ddec54f874bafe1f87a5423f6135222cb002ff0f8e4d0e532433aaaf3cd3c0 you will find a version which supports multiple python versions. Also I am working on a hook which does all of this. When I am done I am going to convert this package, too.
johnnycanencrypt | ||
pyside2 | ||
]; | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add a pythonImportsCheck.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added for johnnycanencrypt, tumpa seems to be lacking tests.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pythonImportsCheck is a replacement for tests to know if the module can be imported at all. Please add this.
LIBCLANG_PATH = llvmPackages.libclang + "/lib"; | ||
|
||
propagatedBuildInputs = [ | ||
python |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That is a nativeBuildInput of maturin to check if python is available, right?
This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch) Result of 6 packages built:
The following issues got detected with the above build packages. python37Packages.tumpa: Zero tests run by pytest got detected: 'Ran 0 tests in 0.000s'
Zero tests run by pytest got detected: 'Ran 0 tests in 0.000s'
Zero tests run by pytest got detected: 'Ran 0 tests in 0.000s'
|
''; | ||
postCheck = "popd"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
''; | |
postCheck = "popd"; | |
''; | |
postCheck = '' | |
popd | |
''; |
This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch) Result of 6 packages built:
|
darwin diff for johnny diff --git a/pkgs/development/python-modules/johnnycanencrypt/default.nix b/pkgs/development/python-modules/johnnycanencrypt/default.nix
index 607c36c8da7..407bb44d8b2 100644
--- a/pkgs/development/python-modules/johnnycanencrypt/default.nix
+++ b/pkgs/development/python-modules/johnnycanencrypt/default.nix
@@ -13,6 +13,7 @@
, numpy
, pytestCheckHook
, pythonOlder
+, PCSC
}:
rustPlatform.buildRustPackage rec {
@@ -48,7 +49,7 @@ rustPlatform.buildRustPackage rec {
buildInputs = [
pcsclite
nettle
- ];
+ ] ++ stdenv.lib.optionals stdenv.isDarwin [ PCSC ];
# Needed b/c need to check AFTER python wheel is installed (using Rust Build, not buildPythonPackage)
doCheck = false;
diff --git a/pkgs/top-level/python-packages.nix b/pkgs/top-level/python-packages.nix
index 75754ba26e0..935155c112d 100644
--- a/pkgs/top-level/python-packages.nix
+++ b/pkgs/top-level/python-packages.nix
@@ -3214,7 +3214,9 @@ in {
joblib = callPackage ../development/python-modules/joblib { };
- johnnycanencrypt = callPackage ../development/python-modules/johnnycanencrypt { };
+ johnnycanencrypt = callPackage ../development/python-modules/johnnycanencrypt {
+ inherit (pkgs.darwin.apple_sdk.frameworks) PCSC;
+ };
josepy = callPackage ../development/python-modules/josepy { };
Tumpa does not work on darwin due to some unrelated build failure. |
Motivation for this change
Add package tumpa with dependency johnnycanencrypt.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)