Skip to content

Build and test the port on Windows in CI - #60

Merged
Adubbz merged 1 commit into
TheMoonPeople:portfrom
carrdylan52:pr/windows-ci-20261005
Oct 6, 2026
Merged

Adubbz merged 1 commit into
TheMoonPeople:portfrom
carrdylan52:pr/windows-ci-20261005

Conversation

@carrdylan52

Copy link
Copy Markdown
Contributor

Adds a windows job to pc.yml, next to the Linux and macOS ones. It builds with tools/windows' scripts and runs the suite apart from the renderer's cases.

  • The runner has no GPU, so there is no software driver here: the GfxMisc and TexDecode cases (the twelve that need a Vulkan device, and one that does not) are left to the Linux and macOS jobs, and darkcloud is not started. Lavapipe on Windows was tried first; its cases were unreliable there.
  • The tests import vulkan-1.dll and do not start without a loader, which the runner may lack. acquire.ps1 -VulkanLoader fetches LunarG's 1.4.363 loader (SHA-256 pinned, like the other archives), configure.ps1 -VulkanLibrary links against it, and the job copies it beside the executables. Without the switches the scripts behave as before.

Testing

The job's steps run locally from a fresh clone and an empty dependency directory, with no Vulkan driver reachable: the build succeeds and 88 of 88 cases pass, three runs in a row. The job itself has not run on GitHub's runner until this PR.

The runner has no GPU, so the job runs the cases that need no Vulkan
device; the renderer's run on Linux and macOS. acquire.ps1 -VulkanLoader
fetches the loader the tests need to start, and configure.ps1 links
against it.
@Adubbz
Adubbz merged commit 86d79b0 into TheMoonPeople:port Oct 6, 2026
9 checks passed
Adubbz pushed a commit that referenced this pull request Oct 6, 2026
The runner has no GPU, so the job runs the cases that need no Vulkan
device; the renderer's run on Linux and macOS. acquire.ps1 -VulkanLoader
fetches the loader the tests need to start, and configure.ps1 links
against it.
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