https://github.com/Kitware/CMake
Revision c5b5bb270f54dab4ff529688a26ea8aa271fa0e1 authored by Brad King on 13 September 2017, 13:29:15 UTC, committed by Brad King on 13 September 2017, 13:37:30 UTC
Since commit v3.9.0-rc1~309^2 (include_external_msproject: Honor
MAP_IMPORTED_CONFIG_<CONFIG>, 2017-04-04) we accidentally honor
`MAP_IMPORTED_CONFIG_<CONFIG>` while generating the `.sln` file entries
for normal targets.  This causes `devenv.com`-driven builds to use the
mapping incorrectly for normal targets.  Check that a target really
comes from `include_external_msproject` before considering the map.

Furthermore, when we do use the map, we should only take the first entry
if more than one configuration is specified.  Otherwise we end up giving
VS a configuration name with a `;` in it.

Fixes: #17276
1 parent ffa1647
History
Tip revision: c5b5bb270f54dab4ff529688a26ea8aa271fa0e1 authored by Brad King on 13 September 2017, 13:29:15 UTC
VS: Do not consider MAP_IMPORTED_CONFIG_<CONFIG> on non-imported targets
Tip revision: c5b5bb2
File Mode Size
Auxiliary
Help
Licenses
Modules
Packaging
Source
Templates
Tests
Utilities
.clang-format -rw-r--r-- 217 bytes
.clang-tidy -rw-r--r-- 806 bytes
.gitattributes -rw-r--r-- 1.1 KB
.gitignore -rw-r--r-- 64 bytes
.hooks-config -rw-r--r-- 418 bytes
CMakeCPack.cmake -rw-r--r-- 9.5 KB
CMakeCPackOptions.cmake.in -rw-r--r-- 12.6 KB
CMakeGraphVizOptions.cmake -rw-r--r-- 153 bytes
CMakeLists.txt -rw-r--r-- 31.0 KB
CMakeLogo.gif -rw-r--r-- 4.4 KB
CONTRIBUTING.rst -rw-r--r-- 1.8 KB
CTestConfig.cmake -rw-r--r-- 440 bytes
CTestCustom.cmake.in -rw-r--r-- 6.1 KB
CompileFlags.cmake -rw-r--r-- 3.2 KB
Copyright.txt -rw-r--r-- 4.9 KB
DartConfig.cmake -rw-r--r-- 374 bytes
README.rst -rw-r--r-- 2.9 KB
bootstrap -rwxr-xr-x 47.2 KB
cmake_uninstall.cmake.in -rw-r--r-- 790 bytes
configure -rwxr-xr-x 99 bytes
doxygen.config -rw-r--r-- 27.4 KB

README.rst

back to top