[Cmake-commits] CMake branch, master, updated. v3.12.1-516-g5563d6b

Kitware Robot kwrobot at kitware.com
Thu Aug 30 09:35:03 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  5563d6b3a64f6c9a83c1ab928e2283c84cb4af41 (commit)
       via  fb73edf70cc1aea44eac27c8827e616c650bbe7d (commit)
       via  bd7dd6d414303e8fc4da786107f4292768d87638 (commit)
      from  b11e021bed6d406e3a7b9af6f5ef8fa5565881af (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=5563d6b3a64f6c9a83c1ab928e2283c84cb4af41
commit 5563d6b3a64f6c9a83c1ab928e2283c84cb4af41
Merge: b11e021 fb73edf
Author:     Brad King <brad.king at kitware.com>
AuthorDate: Thu Aug 30 13:25:46 2018 +0000
Commit:     Kitware Robot <kwrobot at kitware.com>
CommitDate: Thu Aug 30 09:25:51 2018 -0400

    Merge topic 'minor-cleanup'
    
    fb73edf70c cmake: Drop unnecessary working mode checks
    bd7dd6d414 cmake: Declare VariableWatch member only when used
    
    Acked-by: Kitware Robot <kwrobot at kitware.com>
    Merge-request: !2326


https://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=fb73edf70cc1aea44eac27c8827e616c650bbe7d
commit fb73edf70cc1aea44eac27c8827e616c650bbe7d
Author:     Brad King <brad.king at kitware.com>
AuthorDate: Fri Aug 24 08:15:51 2018 -0400
Commit:     Brad King <brad.king at kitware.com>
CommitDate: Wed Aug 29 09:38:32 2018 -0400

    cmake: Drop unnecessary working mode checks
    
    In `cmake::Run` we return before `Configure` or `Generate` when
    not working in normal mode so they do not need mode checks.
    
    While at it, remove a long-outdated comment about when to save the
    cache.

diff --git a/Source/cmake.cxx b/Source/cmake.cxx
index 5bb1f22..1bf8f7d 100644
--- a/Source/cmake.cxx
+++ b/Source/cmake.cxx
@@ -1262,10 +1262,7 @@ int cmake::ActualConfigure()
   this->UpdateConversionPathTable();
   this->CleanupCommandsAndMacros();
 
-  int res = 0;
-  if (this->GetWorkingMode() == NORMAL_MODE) {
-    res = this->DoPreConfigureChecks();
-  }
+  int res = this->DoPreConfigureChecks();
   if (res < 0) {
     return -2;
   }
@@ -1431,11 +1428,8 @@ int cmake::ActualConfigure()
       "CMakeLists.txt ?");
   }
 
-  // only save the cache if there were no fatal errors
-  if (this->GetWorkingMode() == NORMAL_MODE) {
-    this->State->SaveVerificationScript(this->GetHomeOutputDirectory());
-    this->SaveCache(this->GetHomeOutputDirectory());
-  }
+  this->State->SaveVerificationScript(this->GetHomeOutputDirectory());
+  this->SaveCache(this->GetHomeOutputDirectory());
   if (cmSystemTools::GetErrorOccuredFlag()) {
     return -1;
   }
@@ -1588,7 +1582,7 @@ int cmake::Run(const std::vector<std::string>& args, bool noconfigure)
   }
 
   int ret = this->Configure();
-  if (ret || this->GetWorkingMode() != NORMAL_MODE) {
+  if (ret) {
 #if defined(CMAKE_HAVE_VS_GENERATORS)
     if (!this->VSSolutionFile.empty() && this->GlobalGenerator) {
       // CMake is running to regenerate a Visual Studio build tree
@@ -1634,9 +1628,8 @@ int cmake::Generate()
   // Save the cache again after a successful Generate so that any internal
   // variables created during Generate are saved. (Specifically target GUIDs
   // for the Visual Studio and Xcode generators.)
-  if (this->GetWorkingMode() == NORMAL_MODE) {
-    this->SaveCache(this->GetHomeOutputDirectory());
-  }
+  this->SaveCache(this->GetHomeOutputDirectory());
+
   return 0;
 }
 

https://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=bd7dd6d414303e8fc4da786107f4292768d87638
commit bd7dd6d414303e8fc4da786107f4292768d87638
Author:     Brad King <brad.king at kitware.com>
AuthorDate: Fri Aug 24 07:31:01 2018 -0400
Commit:     Brad King <brad.king at kitware.com>
CommitDate: Wed Aug 29 09:38:32 2018 -0400

    cmake: Declare VariableWatch member only when used

diff --git a/Source/cmake.h b/Source/cmake.h
index 86e06df..4b4c67c 100644
--- a/Source/cmake.h
+++ b/Source/cmake.h
@@ -297,8 +297,10 @@ public:
   ///! this is called by generators to update the progress
   void UpdateProgress(const char* msg, float prog);
 
+#if defined(CMAKE_BUILD_WITH_CMAKE)
   ///! Get the variable watch object
   cmVariableWatch* GetVariableWatch() { return this->VariableWatch; }
+#endif
 
   void GetGeneratorDocumentation(std::vector<cmDocumentationEntry>&);
 
@@ -493,8 +495,6 @@ protected:
 
   void GenerateGraphViz(const char* fileName) const;
 
-  cmVariableWatch* VariableWatch;
-
 private:
   ProgressCallbackType ProgressCallback;
   void* ProgressCallbackClientData;
@@ -525,6 +525,10 @@ private:
   std::string GraphVizFile;
   InstalledFilesMap InstalledFiles;
 
+#if defined(CMAKE_BUILD_WITH_CMAKE)
+  cmVariableWatch* VariableWatch;
+#endif
+
   cmState* State;
   cmStateSnapshot CurrentSnapshot;
   cmMessenger* Messenger;

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

Summary of changes:
 Source/cmake.cxx | 19 ++++++-------------
 Source/cmake.h   |  8 ++++++--
 2 files changed, 12 insertions(+), 15 deletions(-)


hooks/post-receive
-- 
CMake


More information about the Cmake-commits mailing list