-
Notifications
You must be signed in to change notification settings - Fork 780
Add arm64 linux release binaries. NFC #2650
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
Conversation
6afe18b to
fe46342
Compare
sbc100
left a comment
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.
A few more nits regarding naming.
- Can we use
x64rather thanamd64in our public filename - OS should be the first element in the filename
- lets leave the arch out on windows we for now.
.github/workflows/build_release.yml
Outdated
| name: arm64-macos-14 | ||
| - os: windows-latest | ||
| arch: x86_64 | ||
| name: amd64-windows-latest |
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.
Can we just call this one windows for now? (At least until we support more than one windows 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.
name "windows-x64" looks more friendly for developer to integration.
I think it is acceptable ?
sbc100
left a comment
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.
Is there some way we can test this change? Or do we need to wait for the next release to know if it actually works?
e19687b to
adc504d
Compare
| - name: install ninja (linux) | ||
| run: sudo apt-get install ninja-build | ||
| if: startsWith(matrix.os, 'ubuntu-') | ||
| if: contains(matrix.os, 'ubuntu') |
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.
This line can be reverted now I think
sbc100
left a comment
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!
I will do a new release as soon as this lands
|
Hi, @sbc100 |
No I think its fine to just support a single version, but we should build with the oldest version we can for max compatabiliy. Folks to obscure/old linux distros and just rely on their distro package manager or build it themselves. I did just notices that this PR changed ubuntu-20.04 to ubuntu-latest. I failed to notice that in review. Can you revert that part please? We should use |
Sorry for the mistake. Have created a fix patch: |


Support ARM64 builds and improve package naming in the GitHub Actions release workflow.
1. Added ARM64 Support
ubuntu-24.04-arm64runner for native Linux ARM64 compilationmacos-14runner which provides native macOS ARM64 (Apple Silicon) compilationaarch64-ubuntu2. Updated Package Naming
ubuntu-20.04tox86-ubuntufor clarity3. Matrix Configuration
The build matrix now includes:
amd64-ubuntu(ubuntu-20.04)arm64-ubuntu(ubuntu-24.04-arm64)arm64-macos-14(macOS Apple Silicon)amd64-windows-latest(Windows-lastest)Release Artifacts
After these changes, each release will include:
wabt-{version}-amd64-ubuntu.tar.gz- Linux x86_64 buildwabt-{version}-arm64-ubuntu.tar.gz- Linux ARM64 buildwabt-{version}-arm64-macos-14.tar.gz- macOS ARM64 build (Apple Silicon)wabt-{version}-amd64-windows-latest.tar.gz- Windows x86_64 buildwabt-{version}-wasi.tar.gz- WASI build