Skip to content

Add MinGW GCC CI coverage#2597

Open
danyalahmed1995 wants to merge 1 commit into
KhronosGroup:mainfrom
danyalahmed1995:ci/add-mingw-coverage
Open

Add MinGW GCC CI coverage#2597
danyalahmed1995 wants to merge 1 commit into
KhronosGroup:mainfrom
danyalahmed1995:ci/add-mingw-coverage

Conversation

@danyalahmed1995

Copy link
Copy Markdown
Contributor

Closes #2584.

Adds a dedicated Windows x64 MinGW GCC CI job using MSYS2 UCRT64.

The job:

  • installs the MinGW GCC toolchain through MSYS2
  • verifies the selected compiler and target
  • generates the Vulkan-Hpp headers
  • builds the samples and tests with C++23 module support enabled
  • runs the test suite

Fork validation

The validation run completed with 27 passing checks and two failures:

The MinGW failure is intentionally left visible as a toolchain signal. As discussed in #2596, the existing Vulkan-Hpp implementation is valid and links successfully with MSVC and Clang, so this PR does not add a project-side workaround for the MinGW GCC bug.

Refs #2468.

@sharadhr

Copy link
Copy Markdown
Contributor

@asuessenbach that particular CI is meant to fail; see #2468. I think it is ready for merge otherwise.

@asuessenbach

Copy link
Copy Markdown
Contributor

that particular CI is meant to fail; see #2468.

I know. And I'll merge it as soon as the root cause of failure is resolved.

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.

Set up 32-bit Linux CI action and MinGW action

3 participants