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
Parity: ethereum client #35263
Parity: ethereum client #35263
Conversation
version = "1.9.3"; | ||
sha256 = "19qyp6kafnnfhdnbq745v8zybnqizjzzc3k4701ly9hf0dvx53ka"; | ||
cargoSha256 = "1vdvqs6ligp5fkw5s7v44vwqwz5dqa0ipilx0piz6swz0drilima"; | ||
patches = [ ./patches/vendored-sources-1.9.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.
Maybe I should just accept #34034
Probably we could add a flag to explicitly unlock git repo support. Then we know, which packages we need to fix in case there breakage in cargo-vendor
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.
It will be cool to drop this patches, but as I see PR is WIP, is it will merged in near time?
yeah, I have an idea on how to solve this. But it can be brought up later: |
Failure on x86_64-darwin (full log) Partial log (click to expand)
|
description = "Fast, light, robust Ethereum implementation"; | ||
homepage = http://parity.io; | ||
license = licenses.gpl3; | ||
maintainers = [ maintainers.akru ]; |
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.
Can you specify platforms here? In doubt only add those, where you have tested it actually.
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.
Yes, set linux.
Success on x86_64-linux (full log) Partial log (click to expand)
|
Success on aarch64-linux (full log) Partial log (click to expand)
|
Hello, @Mic92! |
Oh, thank you @Mic92! |
Motivation for this change
Added two versions of Parity ethereum client (https://parity.io):
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)