Skip to content

Add Linux arm build to CI and release - #1345

Merged
Prashanth Mundkur (pmundkur) merged 3 commits into
riscv:masterfrom
jordancarlin:add_arm_ci
Oct 15, 2025
Merged

Add Linux arm build to CI and release#1345
Prashanth Mundkur (pmundkur) merged 3 commits into
riscv:masterfrom
jordancarlin:add_arm_ci

Conversation

@jordancarlin

@jordancarlin Jordan Carlin (jordancarlin) commented Oct 15, 2025

Copy link
Copy Markdown
Collaborator

Now that there is a Linux Arm build of the Sail compiler, we should release one of the model too. This helps when running a Linux docker image on an Arm (Apple Silicon) Mac.

@pmundkur

Copy link
Copy Markdown
Collaborator

The cache in sail-setup would now need to be indexed on ${{ runner.arch }} along with ${{ runner.os }}

@jordancarlin

Copy link
Copy Markdown
Collaborator Author

The cache in sail-setup would now need to be indexed on ${{ runner.arch }} along with ${{ runner.os }}

Good catch. More problematically, it seems like the setup-cmake action is not smart enough to download an arm binary of CMake:

/home/runner/work/_temp/6863df02-698f-493d-8b1e-1a34a2fce50a.sh: line 3: /opt/hostedtoolcache/cmake/4.1.2/arm64/cmake-4.1.2-linux-x86_64/bin/cmake: cannot execute binary file: Exec format error

We'll need to switch to manually installing the desired version of CMake instead.

@Timmmm

Copy link
Copy Markdown
Collaborator

Could we fix that action? Might not be too hard.

@jordancarlin

Copy link
Copy Markdown
Collaborator Author

Could we fix that action? Might not be too hard.

There has been nothing merged to that repo in over a year and there are 2 existing attempts to add support that both ran into issues (jwlawson/actions-setup-cmake#74 and jwlawson/actions-setup-cmake#78).

I went ahead and switched to a simple curl download instead. The only downside is that we have to manually maintain what latest means now.

@github-actions

Copy link
Copy Markdown

Test Results

2 115 tests  ±0   2 115 ✅ ±0   20m 44s ⏱️ +53s
    1 suites ±0       0 💤 ±0 
    1 files   ±0       0 ❌ ±0 

Results for commit 4b82d25. ± Comparison against base commit 262244a.

@Timmmm

Copy link
Copy Markdown
Collaborator

Ok that seems like a better option.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Nice!

@pmundkur Prashanth Mundkur (pmundkur) added the will be merged Scheduled to be merged soon if nobody objects label Oct 15, 2025
@pmundkur
Prashanth Mundkur (pmundkur) added this pull request to the merge queue Oct 15, 2025
Merged via the queue into riscv:master with commit 2fa4418 Oct 15, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

will be merged Scheduled to be merged soon if nobody objects

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants