Skip to content

.github/workflows: resolve QEMU 10.x packages more flexibly#444

Merged
hajimehoshi merged 3 commits intoebitengine:mainfrom
tmc:qemu-ci-fix-20260420
Apr 21, 2026
Merged

.github/workflows: resolve QEMU 10.x packages more flexibly#444
hajimehoshi merged 3 commits intoebitengine:mainfrom
tmc:qemu-ci-fix-20260420

Conversation

@tmc
Copy link
Copy Markdown
Contributor

@tmc tmc commented Apr 20, 2026

What issue is this addressing?

Closes #443

What type of issue is this addressing?

bug

What this PR does | solves

Address ci instability regarding qemu release versions.

Copilot AI review requested due to automatic review settings April 20, 2026 20:24
Copy link
Copy Markdown
Contributor

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 updates the CI workflow to avoid hardcoding an exact Debian QEMU package version (which can change over time), by dynamically resolving the latest available QEMU 10.x package during the workflow run.

Changes:

  • Replace QEMU_VERSION with QEMU_MAJOR to stop pinning an exact Debian package version.
  • Discover and install the latest matching qemu-user .deb from the Debian pool at runtime (for both ARM64 and AMD64 runners).

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

Comment thread .github/workflows/test.yml Outdated
Copy link
Copy Markdown
Contributor

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.


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

Comment thread .github/workflows/test.yml Outdated
Comment thread .github/workflows/test.yml Outdated
Copy link
Copy Markdown
Member

@hajimehoshi hajimehoshi left a comment

Choose a reason for hiding this comment

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

LGTM

@TotallyGamerJet Please take a look

Copy link
Copy Markdown
Collaborator

@TotallyGamerJet TotallyGamerJet left a comment

Choose a reason for hiding this comment

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

LGTM

@hajimehoshi hajimehoshi merged commit 56b2dd8 into ebitengine:main Apr 21, 2026
42 checks passed
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.

ci sensitive to qemu release numbers

4 participants