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
python3Packages.diskcache: 4.1.0 -> 5.1.0 #109362
Conversation
@@ -28,8 +28,6 @@ buildPythonPackage rec { | |||
mock | |||
]; | |||
|
|||
disabled = lib.versionAtLeast django.version "2.0"; | |||
|
|||
checkPhase = '' | |||
pytest |
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 switch to pytestCheckHook.
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.
done (I think). Tests pass + dependencies still work.
Result of 5 packages built:
|
This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch) Result of 5 packages built:
|
This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch) Result of 5 packages failed to build and are new build failures:
|
61f8174
to
2811a57
Compare
I don't have darwin to test but I can disable tests on darwin maybe? |
This is an issue with the darwin sandbox.
Yeah sure, go ahead. |
@@ -1,7 +1,7 @@ | |||
{ lib |
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.
{ lib | |
{ stdenv | |
, lib |
@teh please fix the merge conflict. |
FYI: 5.2.1 (https://github.com/grantjenks/python-diskcache/releases/tag/v5.2.1) was released |
This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch). Result of 5 packages built:
|
Motivation for this change
update + unbreak on python3.8+
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)