-
-
Notifications
You must be signed in to change notification settings - Fork 14k
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
go-ethereum: 1.13.5 -> 1.14.3 #311188
go-ethereum: 1.13.5 -> 1.14.3 #311188
Conversation
Result of 3 packages failed to build:
|
Looks like ofBorg gets the build error for linux too: https://logs.ofborg.org/?attempt_id=4ada5d40-bb2c-4c07-91b0-6723f62b51cd&key=nixos%2Fnixpkgs.311188 I'm seeing the same hash on aarch64-darwin:
Was the hash perhaps updated before proxyVendor was set to true? |
Thanks for the report @RaghavSood ! Output tested, PR updated. |
Result of 3 packages built:
|
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, will merge after ofborg builds are done
Strange failure for x86_64-darwing, will check that out shortly |
@GrahamcOfBorg build go-ethereum |
@RaghavSood Looks like the x86_64-darwing build on Ofborg did pass ? |
Both aarch64-darwin and x86 are failing: https://logs.ofborg.org/?key=nixos/nixpkgs.311188&attempt_id=7d3e8ad6-f207-45e9-8375-8c55033f554e |
So the failure actually comes from nixosTests, not the build - but also intermittently - merging first, I will check out the tests soon |
Likely related to #294725 - would explain why it failed, then passed - subsequent runs probably had the test build from the previous round of linux tests available in the cache and could download and run it |
Description of changes
See https://github.com/ethereum/go-ethereum/releases/.
proxyVendor = true
was needed to avoid the following error :Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.