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.block-io: fix incorrect dependency version #56462
Conversation
version = "3.6.6"; | ||
sha256 = "b3cb4af317d9b84f6df50f0cfa6840ba69556af637a83fd971537823e13d601a"; | ||
}; | ||
})) |
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.
We can't do that for Python packages. You should instead patch the source.
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 mean make it work with the latest version of pycryptodome
?
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.
Exactly. I guess the only thing that's necessary for that is patching setup.py.
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 tested it with the last pycryptodome version and opened a PR upstream, this PR is updated with the corresponding patch
e18ffdb
to
f0b211d
Compare
@@ -9,6 +9,13 @@ buildPythonPackage rec { | |||
sha256 = "4909d58b32ab7f93d3cd83fa4bbe4edef42ab7566f016bdb6a405a0d8b1907c9"; | |||
}; | |||
|
|||
patches = [ | |||
(fetchpatch { | |||
url = "https://patch-diff.githubusercontent.com/raw/BlockIo/block_io-python/pull/18.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://patch-diff.githubusercontent.com/raw/BlockIo/block_io-python/pull/18.patch"; | |
url = "https://github.com/BlockIo/block_io-python/commit/8ec0a0cd1afcfcb439d6dc306d8497dd9e4a0993.patch"; |
The content of a PR can change.
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 are right, fixed
f0b211d
to
5032ce4
Compare
@dotlambda could you backport it to 19.03 ? |
(cherry picked from commit 64a63e5)
done in 0ef2bff |
Thanks ! |
Motivation for this change
This should also be backported to 19.03 to fix the build
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)