make.bat: cache the vcredist directory.
authorRay Molenkamp <github@lazydodo.com>
Tue, 5 Jun 2018 13:25:37 +0000 (07:25 -0600)
committerRay Molenkamp <github@lazydodo.com>
Tue, 5 Jun 2018 13:25:37 +0000 (07:25 -0600)
When run from make.bat the environment is setup correctly and the VCToolsRedistDir environment variable exists, on later invocations of cmake this may no longer be the case and a warning was emitted about the missing runtime. we can't rely on InstallRequiredSystemLibraries.cmake here since it uses the compiler version to figure out the correct location and it doesn't know how to deal with clang.

build_files/cmake/platform/platform_win32.cmake

index 6e049c0048f737b6b52c0cedbe3ccaaadddc92c0..ec70f11ea2235c852ae96cc58e36f9dedf313d25 100644 (file)
@@ -31,7 +31,8 @@ endif()
 
 if(CMAKE_C_COMPILER_ID MATCHES "Clang")
        set(MSVC_CLANG On)
-       set(MSVC_REDIST_DIR $ENV{VCToolsRedistDir})
+       set(VC_TOOLS_DIR $ENV{VCToolsRedistDir} CACHE STRING "Location of the msvc redistributables")
+       set(MSVC_REDIST_DIR ${VC_TOOLS_DIR})
        if (DEFINED MSVC_REDIST_DIR)
                file(TO_CMAKE_PATH ${MSVC_REDIST_DIR} MSVC_REDIST_DIR)
        else()