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.pysnow: fix build & tests #97716
Conversation
# for the actual source. the github source is in painful poetry format. | ||
(fetchPypi { | ||
inherit pname version; | ||
sha256 = "5df61091470e48b5b3a6ea75637f69d3aacae20041487ea457a9a0e3093fba8c"; | ||
}) | ||
# for the tests - important because we make some dependency version subsitutions |
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.
you should be able to just use:
format = "pyproject";
and it should be able to build from source fine
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.
oh thats what pyproject is for...
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.
ah so it does work that way, but it has the side effect of disabling it for py27.
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.
I think that's fine, python2 has been EOL for 9 months now
fcec088
to
9592a9e
Compare
ok still works, linux & macos. |
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.
LGTM
Result of nixpkgs-review pr 97716 1
2 packages built:
- python37Packages.pysnow
- python38Packages.pysnow
Motivation for this change
ZHF: #97479
Ok so I do something slightly unusual here.
Because I make some requirements-version substitutions I feel it's important we have the tests running. The tests aren't included in the pypi tarball. Usually this would make me switch to the github archive for the source, but being a poetry project, it's not in a ready-to-go format. So I fetch both the pypi tarball and the github archive, from which I extract the tests.
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)