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.

Sign up for GitHub

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

Jump to bottom

Fix support to latest version of RELIC #207

Merged
merged 6 commits into from
Jun 10, 2021
Merged

Fix support to latest version of RELIC #207

merged 6 commits into from
Jun 10, 2021

Conversation

dfaranha
Copy link
Contributor

@dfaranha dfaranha commented Apr 21, 2021 •
edited
Loading

Minor fixes to take into account recent changes in RELIC:

  • Rename master branch to main
  • Remove unnecessary include of relic_test.h
  • Replaced recently renamed bn_init with something simpler (assuming that ALLOC == AUTO remains in place)

@dfaranha dfaranha mentioned this pull request Apr 21, 2021
Closed
@dfaranha dfaranha mentioned this pull request Jun 10, 2021
Closed
hoffmang9
hoffmang9 approved these changes Jun 10, 2021
View reviewed changes
Copy link
Member

@hoffmang9 hoffmang9 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This got caught in launch and then pooling focus. I'll be happy to have the nightly's passing after we get this merged. Thanks again!

@hoffmang9 hoffmang9 merged commit e3d9ad0 into Chia-Network:main Jun 10, 2021
@emlowe emlowe mentioned this pull request Jun 14, 2021
Closed
UdjinM6 pushed a commit to UdjinM6/bls-signatures that referenced this pull request Dec 1, 2021
@dfaranha @UdjinM6
Fix support to latest version of RELIC (Chia-Network#207)
5290e1d 
* Update to RELIC's change from COMP to CFLAGS.

* Rename RELIC master branch to main.

* Remove RELIC include that creates conflicts with TEST_CASE macro.

* Remove bn_init from private_key.cpp because AUTO is enforced in the build.

* Fix the BN size.

* Update Readme to force CI rerun
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hoffmang9 hoffmang9 hoffmang9 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dfaranha @hoffmang9