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

Enable CGO to address DNS issues #213

Merged
merged 1 commit into from
Feb 10, 2022
Merged

Conversation

sudermanjr
Copy link
Member

This PR fixes issue #206

Checklist

  • I have signed the CLA
  • I have updated/added any relevant documentation

Description

What's the goal of this PR?

Seems that by disabling CGO, we broke some DNS resolution stuff that is ever so annoying in Go

What changes did you make?

Removed the CGO_ENABLED=false env var from goreleaser

What alternative solution should we consider, if any?

This doesn't seem to hurt anything, so I say go for it.

@sudermanjr sudermanjr merged commit 2a901e7 into master Feb 10, 2022
@sudermanjr sudermanjr deleted the sudermanjr/enable-cgo branch February 10, 2022 20:06
@Nuru Nuru mentioned this pull request Feb 21, 2022
2 tasks
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