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

Compile static library #173

Draft
wants to merge 7 commits into
base: master
Choose a base branch
from
Draft

Compile static library #173

wants to merge 7 commits into from

Conversation

tothtamas28
Copy link
Contributor

@tothtamas28 tothtamas28 commented Apr 9, 2024

Fixes #152

Related:

Changes:

  • Compiles a static library plugin.a that packages plugin-c/*.o (except plugin-c/blake2-*.o)
  • Packages a static library krypto.a that merges all the static libraries
  • Updates tests to kompile using krypto.a instead of the source files
  • Fixes an issue where plugin-c/*.cpp is compiled before libcryptopp and libff are built
  • Adds -fPIC and -O3 to CPPFLAGS

TODOs before merging:

  • Update flake.nix

@tothtamas28 tothtamas28 self-assigned this Apr 9, 2024
@tothtamas28 tothtamas28 force-pushed the static-lib branch 2 times, most recently from 0f4a680 to 365ff9c Compare April 9, 2024 13:41
@tothtamas28
Copy link
Contributor Author

@Baltoli, @dwightguth, before I proceed with fixing the Nix distribution, can I get a sanity check on the changes so far?

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.

Build library
1 participant