Skip to content

fix: Use standard VGA adapter for legacy Windows - #2243

Merged
kroese merged 1 commit into
masterfrom
dev
Oct 5, 2026
Merged

kroese merged 1 commit into
masterfrom
dev

Conversation

@kroese

@kroese kroese commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI balanced review requested due to automatic review settings October 5, 2026 21:26
@kroese
kroese merged commit 013ea27 into master Oct 5, 2026
3 checks passed
@kroese
kroese deleted the dev branch October 5, 2026 21:27

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

The focused adapter mapping change is internally consistent with existing display handling.

Review effort: Balanced
Findings: None

What changed in this PR

Updates legacy Windows display defaults to use QEMU’s standard VGA adapter.

Changes:

  • Retains VMware VGA for Windows 9x/2000.
  • Uses standard VGA for Windows XP through Windows 8, Server 2003–2012, and ReactOS.
File Description
src/​legacy.sh Adjusts VGA defaults by Windows generation.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

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