-
Notifications
You must be signed in to change notification settings - Fork 136
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
regenerate CGO bindings + drop unused struct which referenced nonexistent abi type #73
Conversation
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.
Works for me
@@ -0,0 +1,122 @@ | |||
{ |
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.
why did you add these into this code base?
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.
why did you add these into this code base?
CircleCI uses this file (see the .circleci/config.yaml in this PR) to download Groth parameters. Previously, we were cargo install
'ing paramcache and generating them ourselves based on a rust-fil-proofs Git SHA, which was a nightmare.
* update api from election post to winning post * add winning post api * generate new CGO bindings * happy clippyg * cleanup and add test for window post * windWOW.bizgit add . * basic adjustments to the go interface * gofmt * fix go tests * fix constants conversions * upate api * update to latest version * regenerate CGO bindings + drop unused struct which referenced non… (#73) * use latest specs-actors * generate new CGO bindings * delete now-dead types * fix warnings from oddly-formed format string * drop EligibleSectors field, which wasn't used / didn't exist * circleci to use go-paramfetch to download (instead of paramcache to generate) * linter said i had to delete you. sorry, toSectors * clippy made me do it * clippy says conversion wasn't necessary * clippy says reading to end of file is naughty * disable pesky gomodules for binary build * so annoying * write a dumb property test for GenerateWinningPoStSectorChallenge to ensure i'm not crazy * add failing property test * copy values from one memory space to the other Co-authored-by: laser <[email protected]>
…in-project#69) * update api from election post to winning post * add winning post api * generate new CGO bindings * happy clippyg * cleanup and add test for window post * windWOW.bizgit add . * basic adjustments to the go interface * gofmt * fix go tests * fix constants conversions * upate api * update to latest version * regenerate CGO bindings + drop unused struct which referenced non… (filecoin-project#73) * use latest specs-actors * generate new CGO bindings * delete now-dead types * fix warnings from oddly-formed format string * drop EligibleSectors field, which wasn't used / didn't exist * circleci to use go-paramfetch to download (instead of paramcache to generate) * linter said i had to delete you. sorry, toSectors * clippy made me do it * clippy says conversion wasn't necessary * clippy says reading to end of file is naughty * disable pesky gomodules for binary build * so annoying * write a dumb property test for GenerateWinningPoStSectorChallenge to ensure i'm not crazy * add failing property test * copy values from one memory space to the other Co-authored-by: laser <[email protected]>
Things in PR
This changeset fixes the CGO compilation errors (mismatches between generated CGO and the header file) and the non-CGO errors, too.
This changeset also replaces the now-antiquated usage of paramcache with go-paramfetch, much like storage-fsm and the like.
It also updates some Rust issues that clippy caught.