Skip to content

ci: Remove QEMU aarch64 build, drop target-cpu=native from repo config#14

Merged
dimitris-norce merged 1 commit intomainfrom
ci/remove-qemu-aarch64-drop-native-cpu
May 4, 2026
Merged

ci: Remove QEMU aarch64 build, drop target-cpu=native from repo config#14
dimitris-norce merged 1 commit intomainfrom
ci/remove-qemu-aarch64-drop-native-cpu

Conversation

@dimitris-norce
Copy link
Copy Markdown
Contributor

Summary

  • Removes build-linux-aarch64 job from both ci.yml and release.yml. The job was running via QEMU software emulation on x86_64 runners, taking 58+ minutes per release. Not worth the wait — no native ARM64 runner was being used.
  • Removes target-cpu=native from .cargo/config.toml. This flag was applying globally including in CI, where it provides no benefit (runners aren't specialized hardware) and can interfere with cross-compilation. Developers who want it locally can add it to ~/.cargo/config.toml.

linux-aarch64 builds via QEMU emulation were taking 58+ minutes.
Removed the job from both ci.yml and release.yml.

target-cpu=native in .cargo/config.toml was applying to CI runners
where it has no benefit and can interfere. Moved it to a local comment
so developers can set it in ~/.cargo/config.toml instead.
@dimitris-norce dimitris-norce marked this pull request as ready for review May 4, 2026 11:23
@dimitris-norce dimitris-norce requested a review from Copilot May 4, 2026 11:23
Copy link
Copy Markdown

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 1 additional finding.

Open in Devin Review

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR removes the emulated Linux ARM64 build from CI/release workflows and stops forcing -Ctarget-cpu=native repo-wide so builds are more portable across CI and cross-compilation environments.

Changes:

  • Removed the QEMU-based build-linux-aarch64 job from CI and release workflows.
  • Updated the release workflow to publish only Linux x86_64, macOS aarch64, and Windows x86_64 artifacts.
  • Replaced the shared Cargo target-cpu=native setting with guidance to configure it only in a developer’s local ~/.cargo/config.toml.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
.github/workflows/release.yml Removes Linux ARM64 release artifact generation and upload.
.github/workflows/ci.yml Removes the Linux ARM64 CI build job.
.cargo/config.toml Drops the repo-wide Rust CPU-specific build flag and leaves local-only guidance.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/release.yml
@dimitris-norce dimitris-norce merged commit 1bdfacb into main May 4, 2026
11 checks passed
@dimitris-norce dimitris-norce deleted the ci/remove-qemu-aarch64-drop-native-cpu branch May 4, 2026 11:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants