-
Notifications
You must be signed in to change notification settings - Fork 62
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
Update deps #992
Update deps #992
Conversation
This patch includes PR GaloisInc/crucible#589. Functions `doFieldStore` and `doFieldLoad` now take a `FieldId` instead of just a `String`.
There's a new failure in |
The failure already happens in 58e511c, the first commit in this PR. |
Bisection on macaw shows that the failure is related to GaloisInc/macaw#183. @chameco, do you have a patch for |
I believe that GaloisInc/aws-lc-verification@7f648ff will resolve this for now, at least. |
This adapts a proof script to work with GaloisInc/macaw#183.
It looks like that revision is working locally for me. I just pushed a patch to the dockerfile to use that revision, so I expect the CI tests to pass now. Thanks, @chameco! Now that there's a reference to it from saw-script, we should make sure that git commit gets merged into the |
Now the blst test has failed, probably due to #991. Perhaps we should merge a fix for that first. |
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.
👍
No description provided.