8000 chore: disable cgo by ym · Pull Request #601 · jetkvm/kvm · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

chore: disable cgo #601

New issue
< 8000 svg height="60" aria-hidden="true" viewBox="0 0 24 24" version="1.1" width="60" data-view-component="true" class="octicon octicon-comment-discussion color-fg-accent">

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

Merged
merged 1 commit into from
Jun 12, 2025
Merged

chore: disable cgo #601

merged 1 commit into from
Jun 12, 2025

Conversation

ym
Copy link
Contributor
@ym ym commented Jun 12, 2025

golang/go#21083

related to #534

@ym ym requested a review from Copilot 8000 June 12, 2025 07:23
Copy link
Contributor
@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR disables CGO for ARM/Linux cross-compilation by updating the build command.

  • Adds CGO_ENABLED=0 to GO_CMD for static binary builds.
Comments suppressed due to low confidence (1)

Makefile:18

  • [nitpick] Add a brief comment explaining why CGO_ENABLED=0 is set (e.g., to produce a statically linked binary) to improve clarity for future maintainers.
GO_CMD := GOOS=linux GOARCH=arm GOARM=7 CGO_ENABLED=0 go

@ym ym merged commit c494cf2 into dev Jun 12, 2025
4 checks passed
ym added a commit that referenced this pull request Jun 12, 2025
@SuperQ
Copy link
Contributor
SuperQ commented Jun 12, 2025

Yay!

ariedel87 pushed a commit to ariedel87/kvm that referenced this pull request Jun 13, 2025
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.

2 participants
0