[Cmake-commits] CMake branch, master, updated. v3.12.2-758-g8fea6b0

Kitware Robot kwrobot at kitware.com
Fri Sep 28 11:05:04 EDT 2018


This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "CMake".

The branch, master has been updated
       via  8fea6b0764dbf1b6893e7ca81d93cddca2facfce (commit)
       via  9c4445a0dd67a0176a00426a3bb6fed149c03810 (commit)
       via  22282d69316eaf6e4a2b50af2ca7585a58f00d28 (commit)
       via  57b9a072cb7e943885a608e99188822b6064708a (commit)
       via  98e4fbdc063ea738f1f2fa38028be14492297d0f (commit)
       via  6597428c364c0a495bba88a6154ac0ed7f3cd461 (commit)
       via  d8294fefe8063cc2e0a0cfecf7fd8fef619cf0da (commit)
      from  8bb0e09e38d3ab75198b1cd9746bfa7a7b80ff94 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
https://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=8fea6b0764dbf1b6893e7ca81d93cddca2facfce
commit 8fea6b0764dbf1b6893e7ca81d93cddca2facfce
Merge: 9c4445a 98e4fbd
Author:     Brad King <brad.king at kitware.com>
AuthorDate: Fri Sep 28 14:58:20 2018 +0000
Commit:     Kitware Robot <kwrobot at kitware.com>
CommitDate: Fri Sep 28 10:58:51 2018 -0400

    Merge topic 'vs-msbuild-platform'
    
    98e4fbdc06 VS: Pass platform when invoking MSBuild
    6597428c36 ctest: Fix generator platform under --build-nocmake
    d8294fefe8 cmake: Fix generator platform under --build mode
    
    Acked-by: Kitware Robot <kwrobot at kitware.com>
    Merge-request: !2413


https://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=9c4445a0dd67a0176a00426a3bb6fed149c03810
commit 9c4445a0dd67a0176a00426a3bb6fed149c03810
Merge: 8bb0e09 22282d6
Author:     Brad King <brad.king at kitware.com>
AuthorDate: Fri Sep 28 14:58:08 2018 +0000
Commit:     Kitware Robot <kwrobot at kitware.com>
CommitDate: Fri Sep 28 10:58:15 2018 -0400

    Merge topic 'addvs2017arm64'
    
    22282d6931 Tests: Add VSWinStore* test for VS 2017 ARM64
    57b9a072cb Tests: Teach VSWinStore* tests to pass the architecture as a parameter
    
    Acked-by: Kitware Robot <kwrobot at kitware.com>
    Merge-request: !2389


https://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=22282d69316eaf6e4a2b50af2ca7585a58f00d28
commit 22282d69316eaf6e4a2b50af2ca7585a58f00d28
Author:     Gilles Khouzam <gillesk at microsoft.com>
AuthorDate: Fri Sep 14 11:33:46 2018 -0700
Commit:     Brad King <brad.king at kitware.com>
CommitDate: Wed Sep 26 06:57:43 2018 -0400

    Tests: Add VSWinStore* test for VS 2017 ARM64

diff --git a/Tests/CMakeLists.txt b/Tests/CMakeLists.txt
index b325039..03f4c32 100644
--- a/Tests/CMakeLists.txt
+++ b/Tests/CMakeLists.txt
@@ -2168,6 +2168,7 @@ ${CMake_BINARY_DIR}/bin/cmake -DDIR=dev -P ${CMake_SOURCE_DIR}/Utilities/Release
       add_test_VSWinStorePhone(vs15-store10_0-X86 "Visual Studio 15 2017" WindowsStore 10.0 Win32)
       add_test_VSWinStorePhone(vs15-store10_0-ARM "Visual Studio 15 2017" WindowsStore 10.0 ARM)
       add_test_VSWinStorePhone(vs15-store10_0-X64 "Visual Studio 15 2017" WindowsStore 10.0 x64)
+      add_test_VSWinStorePhone(vs15-store10_0-ARM64 "Visual Studio 15 2017" WindowsStore 10.0 ARM64)
     endif()
     if(vs14 AND ws10_0)
       add_test_VSWinStorePhone(vs14-store10_0-X86 "Visual Studio 14 2015" WindowsStore 10.0 Win32)

https://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=57b9a072cb7e943885a608e99188822b6064708a
commit 57b9a072cb7e943885a608e99188822b6064708a
Author:     Gilles Khouzam <gillesk at microsoft.com>
AuthorDate: Fri Sep 14 11:33:46 2018 -0700
Commit:     Brad King <brad.king at kitware.com>
CommitDate: Wed Sep 26 06:57:04 2018 -0400

    Tests: Teach VSWinStore* tests to pass the architecture as a parameter
    
    Instead of specifying the architecture in the generator name, pass it as
    the generator platform.  This has been preferred since CMake 3.1.

diff --git a/Tests/CMakeLists.txt b/Tests/CMakeLists.txt
index fb44077..b325039 100644
--- a/Tests/CMakeLists.txt
+++ b/Tests/CMakeLists.txt
@@ -2128,12 +2128,13 @@ ${CMake_BINARY_DIR}/bin/cmake -DDIR=dev -P ${CMake_SOURCE_DIR}/Utilities/Release
 
   get_filename_component(ntver "[HKEY_LOCAL_MACHINE\\Software\\Microsoft\\Windows NT\\CurrentVersion;CurrentVersion]" NAME)
   if(WIN32 AND ntver VERSION_GREATER 6.1) # Windows >= 8.0
-    macro(add_test_VSWinStorePhone name generator systemName systemVersion)
+    macro(add_test_VSWinStorePhone name generator systemName systemVersion architecture)
       add_test(NAME VSWinStorePhone.${name} COMMAND ${CMAKE_CTEST_COMMAND}
         --build-and-test
         "${CMake_SOURCE_DIR}/Tests/VSWinStorePhone"
         "${CMake_BINARY_DIR}/Tests/VSWinStorePhone/${name}"
         --build-generator "${generator}"
+        --build-generator-platform "${architecture}"
         --build-project VSWinStorePhone
         --build-config $<CONFIGURATION>
         --build-options -DCMAKE_SYSTEM_NAME=${systemName}
@@ -2143,14 +2144,14 @@ ${CMake_BINARY_DIR}/bin/cmake -DDIR=dev -P ${CMake_SOURCE_DIR}/Utilities/Release
     endmacro()
 
     if(vs11 AND ws80)
-      add_test_VSWinStorePhone(vs11-store80-X86 "Visual Studio 11 2012" WindowsStore 8.0)
-      add_test_VSWinStorePhone(vs11-store80-ARM "Visual Studio 11 2012 ARM" WindowsStore 8.0)
-      add_test_VSWinStorePhone(vs11-store80-X64 "Visual Studio 11 2012 Win64" WindowsStore 8.0)
+      add_test_VSWinStorePhone(vs11-store80-X86 "Visual Studio 11 2012" WindowsStore 8.0 Win32)
+      add_test_VSWinStorePhone(vs11-store80-ARM "Visual Studio 11 2012" WindowsStore 8.0 ARM)
+      add_test_VSWinStorePhone(vs11-store80-X64 "Visual Studio 11 2012" WindowsStore 8.0 x64)
     endif()
     if(vs12 AND ws81)
-      add_test_VSWinStorePhone(vs12-store81-X86 "Visual Studio 12 2013" WindowsStore 8.1)
-      add_test_VSWinStorePhone(vs12-store81-ARM "Visual Studio 12 2013 ARM" WindowsStore 8.1)
-      add_test_VSWinStorePhone(vs12-store81-X64 "Visual Studio 12 2013 Win64" WindowsStore 8.1)
+      add_test_VSWinStorePhone(vs12-store81-X86 "Visual Studio 12 2013" WindowsStore 8.1 Win32)
+      add_test_VSWinStorePhone(vs12-store81-ARM "Visual Studio 12 2013" WindowsStore 8.1 ARM)
+      add_test_VSWinStorePhone(vs12-store81-X64 "Visual Studio 12 2013" WindowsStore 8.1 x64)
 
       add_test(NAME VSXaml COMMAND ${CMAKE_CTEST_COMMAND}
         --build-and-test
@@ -2164,22 +2165,22 @@ ${CMake_BINARY_DIR}/bin/cmake -DDIR=dev -P ${CMake_SOURCE_DIR}/Utilities/Release
         )
     endif()
     if(CMake_TEST_VSWinStorePhone_VS_2017 AND ws10_0)
-      add_test_VSWinStorePhone(vs15-store10_0-X86 "Visual Studio 15 2017" WindowsStore 10.0)
-      add_test_VSWinStorePhone(vs15-store10_0-ARM "Visual Studio 15 2017 ARM" WindowsStore 10.0)
-      add_test_VSWinStorePhone(vs15-store10_0-X64 "Visual Studio 15 2017 Win64" WindowsStore 10.0)
+      add_test_VSWinStorePhone(vs15-store10_0-X86 "Visual Studio 15 2017" WindowsStore 10.0 Win32)
+      add_test_VSWinStorePhone(vs15-store10_0-ARM "Visual Studio 15 2017" WindowsStore 10.0 ARM)
+      add_test_VSWinStorePhone(vs15-store10_0-X64 "Visual Studio 15 2017" WindowsStore 10.0 x64)
     endif()
     if(vs14 AND ws10_0)
-      add_test_VSWinStorePhone(vs14-store10_0-X86 "Visual Studio 14 2015" WindowsStore 10.0)
-      add_test_VSWinStorePhone(vs14-store10_0-ARM "Visual Studio 14 2015 ARM" WindowsStore 10.0)
-      add_test_VSWinStorePhone(vs14-store10_0-X64 "Visual Studio 14 2015 Win64" WindowsStore 10.0)
+      add_test_VSWinStorePhone(vs14-store10_0-X86 "Visual Studio 14 2015" WindowsStore 10.0 Win32)
+      add_test_VSWinStorePhone(vs14-store10_0-ARM "Visual Studio 14 2015" WindowsStore 10.0 ARM)
+      add_test_VSWinStorePhone(vs14-store10_0-X64 "Visual Studio 14 2015" WindowsStore 10.0 x64)
     endif()
     if(vs11 AND wp80)
-      add_test_VSWinStorePhone(vs11-phone80-X86 "Visual Studio 11 2012" WindowsPhone 8.0)
-      add_test_VSWinStorePhone(vs11-phone80-ARM "Visual Studio 11 2012 ARM" WindowsPhone 8.0)
+      add_test_VSWinStorePhone(vs11-phone80-X86 "Visual Studio 11 2012" WindowsPhone 8.0 Win32)
+      add_test_VSWinStorePhone(vs11-phone80-ARM "Visual Studio 11 2012" WindowsPhone 8.0 ARM)
     endif()
     if(vs12 AND wp81)
-      add_test_VSWinStorePhone(vs12-phone81-X86 "Visual Studio 12 2013" WindowsPhone 8.1)
-      add_test_VSWinStorePhone(vs12-phone81-ARM "Visual Studio 12 2013 ARM" WindowsPhone 8.1)
+      add_test_VSWinStorePhone(vs12-phone81-X86 "Visual Studio 12 2013" WindowsPhone 8.1 Win32)
+      add_test_VSWinStorePhone(vs12-phone81-ARM "Visual Studio 12 2013" WindowsPhone 8.1 ARM)
     endif()
   endif()
 

https://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=98e4fbdc063ea738f1f2fa38028be14492297d0f
commit 98e4fbdc063ea738f1f2fa38028be14492297d0f
Author:     Brad King <brad.king at kitware.com>
AuthorDate: Tue Sep 25 13:25:36 2018 -0400
Commit:     Brad King <brad.king at kitware.com>
CommitDate: Tue Sep 25 13:27:13 2018 -0400

    VS: Pass platform when invoking MSBuild
    
    MSBuild expects a `/p:Platform=...` argument to tell it which platform
    to build among those in the `.vcxproj` files.  We have not historically
    had to do this because we generate only one platform.  However, when
    a project uses `include_external_msproject` the included project file
    may have other platforms.
    
    Fixes: #18308

diff --git a/Source/cmGlobalVisualStudio10Generator.cxx b/Source/cmGlobalVisualStudio10Generator.cxx
index 5ea323a..82fcaad 100644
--- a/Source/cmGlobalVisualStudio10Generator.cxx
+++ b/Source/cmGlobalVisualStudio10Generator.cxx
@@ -938,6 +938,7 @@ void cmGlobalVisualStudio10Generator::GenerateBuildCommand(
     configArg += "Debug";
   }
   makeCommand.push_back(configArg);
+  makeCommand.push_back("/p:Platform=" + this->GetPlatformName());
   makeCommand.push_back(std::string("/p:VisualStudioVersion=") +
                         this->GetIDEVersion());
 

https://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=6597428c364c0a495bba88a6154ac0ed7f3cd461
commit 6597428c364c0a495bba88a6154ac0ed7f3cd461
Author:     Brad King <brad.king at kitware.com>
AuthorDate: Tue Sep 25 13:20:30 2018 -0400
Commit:     Brad King <brad.king at kitware.com>
CommitDate: Tue Sep 25 13:27:11 2018 -0400

    ctest: Fix generator platform under --build-nocmake
    
    When constructing a global generator instance in `--build-and-test` mode
    we need to set the platform passed by `--build-generator-platform`
    directly on the generator.  The old code that set it on the `cmake`
    class instance did nothing because that only affects cache
    initialization, which is not done by `--build-nocmake`.

diff --git a/Source/CTest/cmCTestBuildAndTestHandler.cxx b/Source/CTest/cmCTestBuildAndTestHandler.cxx
index fccbc95..668a387 100644
--- a/Source/CTest/cmCTestBuildAndTestHandler.cxx
+++ b/Source/CTest/cmCTestBuildAndTestHandler.cxx
@@ -5,6 +5,7 @@
 #include "cmCTest.h"
 #include "cmCTestTestHandler.h"
 #include "cmGlobalGenerator.h"
+#include "cmMakefile.h"
 #include "cmSystemTools.h"
 #include "cmWorkingDirectory.h"
 #include "cmake.h"
@@ -210,9 +211,14 @@ int cmCTestBuildAndTestHandler::RunCMakeAndTest(std::string* outstring)
 
   if (this->BuildNoCMake) {
     // Make the generator available for the Build call below.
-    cm.SetGlobalGenerator(cm.CreateGlobalGenerator(this->BuildGenerator));
-    cm.SetGeneratorPlatform(this->BuildGeneratorPlatform);
-    cm.SetGeneratorToolset(this->BuildGeneratorToolset);
+    cmGlobalGenerator* gen = cm.CreateGlobalGenerator(this->BuildGenerator);
+    cm.SetGlobalGenerator(gen);
+    if (!this->BuildGeneratorPlatform.empty()) {
+      cmMakefile mf(gen, cm.GetCurrentSnapshot());
+      if (!gen->SetGeneratorPlatform(this->BuildGeneratorPlatform, &mf)) {
+        return 1;
+      }
+    }
 
     // Load the cache to make CMAKE_MAKE_PROGRAM available.
     cm.LoadCache(this->BinaryDir);

https://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=d8294fefe8063cc2e0a0cfecf7fd8fef619cf0da
commit d8294fefe8063cc2e0a0cfecf7fd8fef619cf0da
Author:     Brad King <brad.king at kitware.com>
AuthorDate: Tue Sep 25 10:08:57 2018 -0400
Commit:     Brad King <brad.king at kitware.com>
CommitDate: Tue Sep 25 13:27:08 2018 -0400

    cmake: Fix generator platform under --build mode

diff --git a/Source/cmake.cxx b/Source/cmake.cxx
index c26a380..889a5fb 100644
--- a/Source/cmake.cxx
+++ b/Source/cmake.cxx
@@ -2467,6 +2467,14 @@ int cmake::Build(int jobs, const std::string& dir, const std::string& target,
       return 1;
     }
   }
+  const char* cachedGeneratorPlatform =
+    this->State->GetCacheEntryValue("CMAKE_GENERATOR_PLATFORM");
+  if (cachedGeneratorPlatform) {
+    cmMakefile mf(gen, this->GetCurrentSnapshot());
+    if (!gen->SetGeneratorPlatform(cachedGeneratorPlatform, &mf)) {
+      return 1;
+    }
+  }
   std::string output;
   std::string projName;
   const char* cachedProjectName =

-----------------------------------------------------------------------

Summary of changes:
 Source/CTest/cmCTestBuildAndTestHandler.cxx | 12 +++++++---
 Source/cmGlobalVisualStudio10Generator.cxx  |  1 +
 Source/cmake.cxx                            |  8 +++++++
 Tests/CMakeLists.txt                        | 36 +++++++++++++++--------------
 4 files changed, 37 insertions(+), 20 deletions(-)


hooks/post-receive
-- 
CMake


More information about the Cmake-commits mailing list