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

[cryptolib] Add support for cryptolib static library release target #21196

Closed
moidx opened this issue Feb 5, 2024 · 2 comments
Closed

[cryptolib] Add support for cryptolib static library release target #21196

moidx opened this issue Feb 5, 2024 · 2 comments
Assignees
Labels
SW:cryptolib Crypto library
Milestone

Comments

@moidx
Copy link
Contributor

moidx commented Feb 5, 2024

Description

The cryptolib is currently built with the following target:

bazel build --platforms=@crt//platforms/riscv32:opentitan //sw/device/lib/crypto:otcrypto

Bazel however does not have support for generating static library binaries by default. There is an RFC proposing the addition of a cc_static_library being tracked in bazelbuild/bazel#1920 (see doc and gist).

This issue is to track the implementation of a static library target.

CC @jadephilipoom for visibility

@moidx moidx added the SW:cryptolib Crypto library label Feb 5, 2024
@moidx moidx added this to the cryptolib milestone Feb 5, 2024
@cfrantz
Copy link
Contributor

cfrantz commented Feb 5, 2024

#21198 adapts one of the gists to our use case.

@moidx
Copy link
Contributor Author

moidx commented Feb 10, 2024

Fixed by #21198

@moidx moidx closed this as completed Feb 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
SW:cryptolib Crypto library
Projects
None yet
Development

No branches or pull requests

2 participants