ci: add riscv64 wheel build via native RISE runner#2
Conversation
Add QEMU emulation for riscv64 and include riscv64 in the build matrix so that linux_riscv64 wheels are built alongside existing architectures.
Signed-off-by: Bruno Verachten <gounthar@gmail.com>
The QEMU setup step was present but the riscv64 matrix entry was missing, so it never triggered. Replace the QEMU approach with a native ubuntu-24.04-riscv runner (RISE Project). The pypa/cibuildwheel action calls actions/setup-python internally, which has no riscv64 binaries. Add a dedicated riscv64 step that installs cibuildwheel via pip and runs it directly, consistent with how other projects handle native riscv64 CI with RISE runners. Signed-off-by: Bruno Verachten <gounthar@gmail.com>
|
Note Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported. |
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughAdded RISC-V CI support: build-wheels workflow gains an Changes
Sequence Diagram(s)mermaid Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
workflow_dispatch trigger that builds the sdist then runs cibuildwheel on the native ubuntu-24.04-riscv runner. Bypasses the full CI pipeline which requires a fork release to exist. Produces real build times for upstream discussion. Signed-off-by: Bruno Verachten <gounthar@gmail.com>
Internal benchmark PR — targeting the fork to get real build times on ubuntu-24.04-riscv before proposing this upstream to jcrist#987.
Upstream does not have the RISE runners app installed yet, so CI numbers need to come from here first.
Summary by CodeRabbit