-
-
Notifications
You must be signed in to change notification settings - Fork 56.5k
CMake: Potential bug (code review) #18490
Copy link
Copy link
Closed
Milestone
Description
Disclaimer: not tested, just reviewing the code...
Since CMake now provide a multi configuration ninja generator, I guess this check is wrong:
Lines 1032 to 1034 in a3e7c2d
| if(NOT CMAKE_GENERATOR MATCHES "Xcode|Visual Studio") | |
| status(" Configuration:" ${CMAKE_BUILD_TYPE}) | |
| endif() |
Fix: please add Ninja Multi-Config (or simply Multi-Config 🤔) to the regexp...
Reactions are currently unavailable