-
Notifications
You must be signed in to change notification settings - Fork 9
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
add arm support #83
add arm support #83
Conversation
.github/workflows/release.yaml
Outdated
- name: Build binaries for ${{ matrix.arch }} | ||
run: | | ||
mkdir -p dist | ||
go build -o dist/myapp-${{ matrix.arch }} . | ||
|
||
- name: Upload binaries | ||
uses: actions/upload-artifact@v3 | ||
with: | ||
name: binaries-${{ matrix.arch }} | ||
path: dist/myapp-${{ matrix.arch }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Goreleaser already builds the binaries and images for all required architectures. A separate Github matrix build is not necessary.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i means that the change is obsolet? Can you help me please to finish adding arm build support?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Depends; if you only need the binaries, your additional entry in .goreleaser.yml
is sufficient. If you need a multi-arch Docker image, this would be a bit more work (look at the .goreleaser.yml
of the mittwald/kubernetes-replicator project for reference).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i only need arm64 binary, i reverted other changes
No description provided.