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

Go required for Rust builds #661

Closed
pront opened this issue Jan 16, 2025 · 3 comments
Closed

Go required for Rust builds #661

pront opened this issue Jan 16, 2025 · 3 comments

Comments

@pront
Copy link

pront commented Jan 16, 2025

Problem:

Go is a build dependency for Rust builds.

Looking at:

if !test_go_command() {

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.

@justsmth
Copy link
Contributor

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! 😊

@pront
Copy link
Author

pront commented Jan 18, 2025

Hi @justsmth, thank you for the quick and comprehensive reply 👍

Removing the Go requirement is something we would like to do, but it might take awhile to get there.

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.

@justsmth
Copy link
Contributor

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.

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

No branches or pull requests

2 participants