-
Notifications
You must be signed in to change notification settings - Fork 278
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
cmake: Only search for MASM when using MSVC #1291
Conversation
Author sfan5 not on autobuild list. Waiting for curator authorization before starting CI build. |
1 similar comment
Author sfan5 not on autobuild list. Waiting for curator authorization before starting CI build. |
CI Vulkan-Loader build queued with queue ID 41269. |
CI Vulkan-Loader build # 2156 running. |
CI Vulkan-Loader build # 2156 passed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Update .github\workflows\build.yml
mingw-no-asm
sets -D USE_MASM=OFF
which shouldn't be needed with this change now.
Author sfan5 not on autobuild list. Waiting for curator authorization before starting CI build. |
1 similar comment
Author sfan5 not on autobuild list. Waiting for curator authorization before starting CI build. |
CI Vulkan-Loader build queued with queue ID 41316. |
CI Vulkan-Loader build # 2158 running. |
CI Vulkan-Loader build # 2158 passed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM but @charles-lunarg should double check
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yup looks fine!
When cross-compiling with MinGW CMake somehow says it finds MASM and the build subsequently fails since gcc doesn't understand
/Fa
or/Od
. While this could possible be a CMake bug I think it's a quick fix here.This shouldn't break anything else as MASM is restricted to MSVC and not available in other situations.
reproduction: same script as #1278 just remove the
-DUSE_MASM=OFF