-
Notifications
You must be signed in to change notification settings - Fork 56
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
Go required for Rust builds #661
Comments
Hello! Go is only needed for building the FIPS module for AWS-LC. Except for applications that will deploy into a FIPS-compliant environment, most consumers should not enable the "fips" feature. For (non-FIPS) builds on most platforms, only a C compiler is needed. We have more details in our User Guide. The reason for the Go requirement has to do with the need to remove "relocations" from the the library so that the module's integrity test can pass as runtime. This logic was implemented in Go. There's more information about this here. Removing the Go requirement is something we would like to do, but it might take awhile to get there. Please let us know if there's a way that we can make the build experience better. Thanks! 😊 |
Hi @justsmth, thank you for the quick and comprehensive reply 👍
Sounds like a worthwhile goal; I created this issue because I wanted to know if there's any effort towards this that I can track. Feel free to close this issue if you want. |
Thanks for making contact about this! I submitted an Issue upstream for AWS-LC to remove its Go requirement for the FIPS build: I'll close the issue here for now since this first needs to be addressed upstream. |
Problem:
Go is a build dependency for Rust builds.
Looking at:
aws-lc-rs/aws-lc-fips-sys/builder/cmake_builder.rs
Line 374 in 9124243
Is there any plan to remove the Go dependency?
Relevant details
I realized Go >= 1.18 is needed while working on https://github.com/vectordotdev/vector/pull/21504/files.
Especially for older Ubuntu versions, we have to rely on an unofficial PPA as described here: https://go.dev/wiki/Ubuntu
I would love to not have to depend on Go.
The text was updated successfully, but these errors were encountered: