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.cytoolz: fix build #51663
Conversation
This will conflict with b5cb0f0, even though your fix looks better. |
@dotlambda Hoopsie, didn't see that commit! Also maybe I should have directed this PR to |
@GrahamcOfBorg build python3Packages.cytoolz |
# temporal fix for a test failure: https://github.com/pytoolz/cytoolz/issues/122 | ||
(fetchpatch { | ||
name = "py37.patch"; | ||
url = "https://salsa.debian.org/python-team/modules/python-cytoolz/raw/master/debian/patches/py37.patch"; |
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.
url = "https://salsa.debian.org/python-team/modules/python-cytoolz/raw/master/debian/patches/py37.patch"; | |
url = "https://salsa.debian.org/python-team/modules/python-cytoolz/raw/5ce4158deefc47475d1e76813f900e6c72ddcc6e/debian/patches/py37.patch"; |
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 of course! Fixed it.
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 also change the URL in the commit messsage.
And a minor nitpick: some people prefer using quotes-free URLs given this is a thing in Nix.
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.
+1 fixed it
d428ec4
to
3854f88
Compare
rebased to current |
@GrahamcOfBorg build python3Packages.cytoolz python2Packages.cytoolz |
94cc8c3
to
d3ef36d
Compare
cytoolz has a test failure with Python 3.7 [0]. There is a debian patch for it, which didn't make it into upstream yet [1]. [0] pytoolz/cytoolz#122 [1] https://salsa.debian.org/python-team/modules/python-cytoolz/raw/5ce4158deefc47475d1e76813f900e6c72ddcc6e/debian/patches/py37.patch
d3ef36d
to
df6010a
Compare
@GrahamcOfBorg build python3Packages.cytoolz python2Packages.cytoolz |
cytoolz has a test failure with Python 3.7. There is a debian patch for it, which didn't make it into upstream yet.
Motivation for this change
python3Packages.cytoolz
does not build onmaster
.Things done
Apply the patch from debian.
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)