Skip to content

[codex] Fix Windows clang CI build#32

Merged
L-Sun merged 1 commit into
devfrom
codex/fix-windows-ci-ccache
May 1, 2026
Merged

[codex] Fix Windows clang CI build#32
L-Sun merged 1 commit into
devfrom
codex/fix-windows-ci-ccache

Conversation

@L-Sun
Copy link
Copy Markdown
Owner

@L-Sun L-Sun commented Apr 28, 2026

Summary

  • Disable ccache/sccache for the Windows clang-cl GitHub Actions job, including the dependency-install phase.
  • Keep the Windows workflow on clang-cl only while MSVC remains disabled.
  • Fix the playground WinPIX loader to use wide-character Windows APIs when UNICODE is defined.

Root cause

The latest windows-build run first failed before project compilation while xmake installed assimp v6.0.4. On windows-latest, ccache.exe from Strawberry Perl wrapped CMake's cmcldeps resource-compiler invocation and caused the RC /fo argument to be interpreted by clang-cl, ending with clang-cl: error: no such file or directory: '/fo'.

After disabling compiler cache launchers, PR CI progressed into the project build and exposed a real clang-cl compile error in examples/playground/main.cpp: with UNICODE enabled, GetModuleHandle and LoadLibrary resolve to the W variants but were passed narrow strings.

Validation

  • xmake f -y -a x64 -m release --toolchain=clang-cl --ccache=n --policies=package.build.ccache:n,build.ccache:n
  • xmake build playground
  • xmake build

This PR is draft so PR CI can validate the GitHub runner path before merging.

@L-Sun L-Sun force-pushed the codex/fix-windows-ci-ccache branch 2 times, most recently from 457a02b to 292183e Compare April 28, 2026 21:29
@L-Sun L-Sun force-pushed the codex/fix-windows-ci-ccache branch from 292183e to c4204b5 Compare April 28, 2026 21:47
@L-Sun L-Sun changed the title [codex] Disable ccache for Windows clang CI [codex] Fix Windows clang CI build Apr 28, 2026
@L-Sun L-Sun marked this pull request as ready for review May 1, 2026 09:31
@L-Sun L-Sun merged commit 3e019a5 into dev May 1, 2026
1 check 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.

1 participant