CXX-2726 do not override CMAKE_BUILD_TYPE for multi-config generator #1006
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
CMAKE_BUILD_TYPE
for multi-config generatorCMAKE_BUILD_TYPE
from Windows install instructions--config
in Windows install instructionsBackground & Motivation
Configuring on Windows with Visual Studio results in this log output from CMake:
The log message is printed when overriding
CMAKE_BUILD_TYPE
. OverridingCMAKE_BUILD_TYPE
is ignored for multi-configuration generators:This may mislead users to think the build is Release, when it may be Debug. This may result in confusing runtime errors if mixing Debug and Release builds in an application. This PR changes to not override
CMAKE_BUILD_TYPE
for multi-configuration generator.See CXX-2726 for user reports motivating this change.
To configure and build on Windows, the following commands were used: