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

Explicitly set CGO_ENABLED=0 #127

Merged
merged 2 commits into from
Apr 23, 2024
Merged

Explicitly set CGO_ENABLED=0 #127

merged 2 commits into from
Apr 23, 2024

Conversation

shawnHartsell
Copy link
Contributor

@shawnHartsell shawnHartsell commented Apr 23, 2024

Relates to #95

Updating script/build and .goreleaser to explicitly set CGO_ENABLED=0. This is required after upgrading to Go >= 1.20.

Even though we're not using CGO directly, some packages in the standard library like net (that we do use) will implicitly use the CGO implementation if the build environment has the C tools available. Starting in Go 1.20 the standard library does not ship pre-compiled archives, so to suppress this behavior CGO must be explicitly disabled.

See golang/go#58550 for more details

In addition I also made minor changes to the .goreleaser.yml config to make it easier to read as well as to force the use of draft releases which is implicitly called out in the release docs

@shawnHartsell shawnHartsell requested review from a team as code owners April 23, 2024 16:18
Copy link
Contributor

@ostrower ostrower left a comment

Choose a reason for hiding this comment

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

Looks great!

@shawnHartsell shawnHartsell merged commit 206dd07 into main Apr 23, 2024
4 checks passed
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.

None yet

2 participants