Skip to content
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

[Crypto] update BLST version #4693

Merged
merged 223 commits into from
Sep 13, 2023
Merged

Conversation

tarakby
Copy link
Contributor

@tarakby tarakby commented Sep 7, 2023

Upgrade the BLST version to tag v0.3.11.
(checklist copied from blst_src/README)

  • audit all BLST updates, with focus on <blst>/src: https://github.com/supranational/blst/compare/<old_version>...<new_version>
  • delete all files in this folder ./blst_src/ but blst_src.c and README.md.
  • delete all files in ./internal/blst/.
  • open BLST repository on the new version.
  • copy all .c and .h files from <blst>/src/ into ./blst_src/.
  • delete newly copied ./blst_src/server.c.
  • copy the folder <blst>/build/ into this folder ./blst_src.
  • copy <blst>/bindings/blst.h, <blst>/bindings/blst_aux.h, and <blst>/bindings/go/blst.go into ./internal/blst/..
  • check that C flags in ./bls12381_utils.go still include the C flags in <blst>/bindings/go/blst.go.
  • update ./blst_src/blst_src.c if needed.
  • solve all breaking changes that may occur.
  • update the commit version on this ./blst_src/README.

The PR also merged the master branch into the feature branch to get the latest test fixes.

zhangchiqing and others added 30 commits June 22, 2023 10:55
Co-authored-by: Peter Argue <89119817+peterargue@users.noreply.github.com>
@tarakby tarakby merged commit 6fb9707 into feature/blst-based-crypto Sep 13, 2023
34 of 36 checks passed
@tarakby tarakby deleted the tarak/blst-update-blst branch September 13, 2023 04:28
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants