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
feat: add arm64 binaries for Windows #3643
Conversation
related to containers#3443 Signed-off-by: Florent Benoit <fbenoit@redhat.com>
- uses: actions/upload-artifact@v3.1.2 | ||
with: | ||
name: windows-exe-x64 | ||
path: ./dist/podman-desktop*-next-x64.exe |
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.
Shouldn't it be ./dist/podman-desktop*-x64.exe
without next
as in the portable artifactName there is no next?
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 wanted to check with the PR
but on my end I have in dist folder
ls -l dist
-rw-r--r--@ 1 benoitf staff 7831 Aug 23 10:14 builder-debug.yml
-rw-r--r--@ 1 benoitf staff 1888 Aug 23 10:12 builder-effective-config.yaml
-rw-r--r--@ 1 benoitf staff 713 Aug 23 10:14 latest.yml
-rw-r--r--@ 1 benoitf staff 83757463 Aug 23 10:14 podman-desktop-1.4.0-next-arm64.exe
-rw-r--r--@ 1 benoitf staff 83930056 Aug 23 10:14 podman-desktop-1.4.0-next-setup-arm64.exe
-rw-r--r--@ 1 benoitf staff 88850 Aug 23 10:14 podman-desktop-1.4.0-next-setup-arm64.exe.blockmap
-rw-r--r--@ 1 benoitf staff 231126266 Aug 23 10:14 podman-desktop-1.4.0-next-setup-x64.exe
-rw-r--r--@ 1 benoitf staff 240231 Aug 23 10:14 podman-desktop-1.4.0-next-setup-x64.exe.blockmap
-rw-r--r--@ 1 benoitf staff 314515309 Aug 23 10:14 podman-desktop-1.4.0-next-setup.exe
-rw-r--r--@ 1 benoitf staff 327191 Aug 23 10:14 podman-desktop-1.4.0-next-setup.exe.blockmap
-rw-r--r--@ 1 benoitf staff 230953372 Aug 23 10:14 podman-desktop-1.4.0-next-x64.exe
-rw-r--r--@ 1 benoitf staff 314342680 Aug 23 10:14 podman-desktop-1.4.0-next.exe
so if I pickup ./dist/podman-desktop*-x64.exe
it means it'll take both the installer and the portable file
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.
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.
LGTM
What does this PR do?
add arm64 binaries for Windows
include the arch in the filename
Screenshot/screencast of this PR
What issues does this PR fix or reference?
related to #3443
How to test this PR?
Execute binaries on arm64 and x84