cmake-init icon indicating copy to clipboard operation
cmake-init copied to clipboard
verified publisher icon friendlyanon

Windows SDK requirement

Open friendlyanon opened this issue 4 years ago • 2 comments

Some Windows SDKs (10.0.17763.0 and earlier?) have bugs in them that emit warnings with the new, standards conforming preprocessor (/Zc:preprocessor). The SDK can be specified by setting the system version for CMake (as is done in a6683d5), but this has potential issues.

Windows developers using this script locally will also have to install a Windows Update newer than 10.0.17763 with the matching Windows SDK to get the correct behavior by default.

This might be too much of a mental overhead for someone just starting out. Should the preprocessor flag be dropped and instead the ubuntu and macos CI runs be used to verify the conformance of the code?

friendlyanon avatar Apr 26 '21 12:04 friendlyanon

I have decided to remove /Zc:preprocessor with 11d35cd to keep things simple. Once the windows-2022 github image is released as the latest one, this could be added back.

friendlyanon avatar Apr 26 '21 14:04 friendlyanon

Once #35 goes through, this can be closed as well.

friendlyanon avatar Dec 15 '21 18:12 friendlyanon

#35 has been closed for a while, so this is not relevant anymore.

friendlyanon avatar Dec 08 '23 13:12 friendlyanon