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.blspy: unbreak #144230
python3Packages.blspy: unbreak #144230
Conversation
e72d7d1
to
6eb1c7d
Compare
|
||
# https://github.com/Chia-Network/bls-signatures/issues/286 | ||
(fetchpatch { | ||
url = "https://github.com/Chia-Network/bls-signatures/commit/797241e9dae1c164c862cbdb38c865d4b124a601.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.
do you mind making this into a PR then taking from the PR. This is only working because of all refs on the upstream repo
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.
There is a PR. If I understand you correctly, I'm not sure this would be better since the PR head is mutable. (and quite likely to 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.
I guess I'm suggesting that the PR ref should be in there as well
url = "https://github.com/Chia-Network/bls-signatures/commit/797241e9dae1c164c862cbdb38c865d4b124a601.patch"; | |
url = "https://github.com/Chia-Network/bls-signatures/pull/287/commits/797241e9dae1c164c862cbdb38c865d4b124a601.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.
good point. done
7fdc177
to
f48d869
Compare
f48d869
to
a956e7b
Compare
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 144230 run on x86_64-linux 1
7 packages built:
- chia
- python38Packages.blspy
- python38Packages.clvm
- python38Packages.clvm-tools
- python39Packages.blspy
- python39Packages.clvm
- python39Packages.clvm-tools
Motivation for this change
#144209
Chia-Network/bls-signatures#286
Things done
Submit patch upstream
sandbox = true
set innix.conf
? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)