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
pythonPackages.pyotp: Disable on Python27 #100023
Conversation
0cbf252
to
ae23d0b
Compare
Could you add a |
ae23d0b
to
639f88a
Compare
Quite new to python packaging, was not aware of this being a thing. Thanks. This showed me the application does not work at all with python 2.7, so I've adjusted the PR accordingly. I've still added an
|
Result of 3 packages built:
|
|
||
src = fetchPypi { | ||
inherit pname version; | ||
sha256 = "01eceab573181188fe038d001e42777884a7f5367203080ef5bda0e30fe82d28"; | ||
}; | ||
|
||
checkInputs = [ urllib3 ]; |
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.
Where does this dependency come from?
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.
Wondering about the urllib3
dependency.
Oops, looks like i accidentally left that in. Removed 👍 |
639f88a
to
7a34808
Compare
Motivation for this change
This disables tests for python27, as they import urllib in an unsupported way.
ZHF: #97479
Result of
nixpkgs-review pr 100023
14 packages built:
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)