[Cmake-commits] CMake branch, master, updated. v3.15.1-543-g09fe6e6
Kitware Robot
kwrobot at kitware.com
Wed Jul 31 11:52:04 EDT 2019
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 09fe6e6e9408dde14193f9a358550ac738581cac (commit)
via 3cf89d18c3f3f2e0fc7478ccd1c483382cdf0a3d (commit)
via ac7ac46fce54ce263dc32cdd12775a12a95814b7 (commit)
via dad7868db812addec83c96c50df106d851d6257b (commit)
via 955ebe1c93a35503bf3c81f25c41e481aa94f439 (commit)
via c81a86ee6d69bbae9f0213765aabc434faf10080 (commit)
via 83368b4dd539f287942e26e53ed49d4ec3a18db0 (commit)
via fd58bb83e6a6dcbe27afd2272d1a9c2722de2482 (commit)
via 800c3ab2e0b5eed54221ac37c634938ca3d273ab (commit)
via 627fc5b44f7c5fab2d6bc81ae3a0d9b06cb9f3bf (commit)
via 7d98cd0442d695df809378dcd5a1eb0c3fb0928a (commit)
via 93af8a2583e2ec8d067a2e8eda544dfc5d3d3b4f (commit)
via 4053d5737d917c3cb34af5d510a65ca26ed1763e (commit)
from cb8863279033d9dd81d01edcbb5cc8ae1911b06c (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=09fe6e6e9408dde14193f9a358550ac738581cac
commit 09fe6e6e9408dde14193f9a358550ac738581cac
Merge: 3cf89d1 c81a86e
Author: Brad King <brad.king at kitware.com>
AuthorDate: Wed Jul 31 15:47:28 2019 +0000
Commit: Kitware Robot <kwrobot at kitware.com>
CommitDate: Wed Jul 31 11:49:23 2019 -0400
Merge topic 'ninja-1.10'
c81a86ee6d Ninja: Record dyndep support by Ninja 1.10
83368b4dd5 Ninja: Drop unused dyndep version check
fd58bb83e6 Ninja: Use in-class initialization of global generator members
Acked-by: Kitware Robot <kwrobot at kitware.com>
Acked-by: Ben Boeckel <ben.boeckel at kitware.com>
Merge-request: !3256
https://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=3cf89d18c3f3f2e0fc7478ccd1c483382cdf0a3d
commit 3cf89d18c3f3f2e0fc7478ccd1c483382cdf0a3d
Merge: ac7ac46 627fc5b
Author: Brad King <brad.king at kitware.com>
AuthorDate: Wed Jul 31 15:46:10 2019 +0000
Commit: Kitware Robot <kwrobot at kitware.com>
CommitDate: Wed Jul 31 11:47:09 2019 -0400
Merge topic 'ExternalProject-avoid-extra-checkout'
627fc5b44f ExternalProject: Avoid unnecessary checkout on clone
Acked-by: Kitware Robot <kwrobot at kitware.com>
Merge-request: !3626
https://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=ac7ac46fce54ce263dc32cdd12775a12a95814b7
commit ac7ac46fce54ce263dc32cdd12775a12a95814b7
Merge: dad7868 800c3ab
Author: Brad King <brad.king at kitware.com>
AuthorDate: Wed Jul 31 15:45:29 2019 +0000
Commit: Kitware Robot <kwrobot at kitware.com>
CommitDate: Wed Jul 31 11:45:38 2019 -0400
Merge topic 'FindPythonInterp-no-version-verify'
800c3ab2e0 FindPythonInterp: Document lack of version filtering for plain `python`
Acked-by: Kitware Robot <kwrobot at kitware.com>
Acked-by: Rolf Eike Beer <eike at sf-mail.de>
Merge-request: !3631
https://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=dad7868db812addec83c96c50df106d851d6257b
commit dad7868db812addec83c96c50df106d851d6257b
Merge: 955ebe1 7d98cd0
Author: Brad King <brad.king at kitware.com>
AuthorDate: Wed Jul 31 11:43:52 2019 -0400
Commit: Brad King <brad.king at kitware.com>
CommitDate: Wed Jul 31 11:43:52 2019 -0400
Merge branch 'release-3.15'
https://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=955ebe1c93a35503bf3c81f25c41e481aa94f439
commit 955ebe1c93a35503bf3c81f25c41e481aa94f439
Merge: cb88632 93af8a2
Author: Brad King <brad.king at kitware.com>
AuthorDate: Wed Jul 31 15:42:11 2019 +0000
Commit: Kitware Robot <kwrobot at kitware.com>
CommitDate: Wed Jul 31 11:43:17 2019 -0400
Merge topic 'source_group-tree-files'
93af8a2583 source_group: Fix regression in relative FILES
Acked-by: Kitware Robot <kwrobot at kitware.com>
Merge-request: !3630
https://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=c81a86ee6d69bbae9f0213765aabc434faf10080
commit c81a86ee6d69bbae9f0213765aabc434faf10080
Author: Brad King <brad.king at kitware.com>
AuthorDate: Tue Jul 30 13:53:44 2019 -0400
Commit: Brad King <brad.king at kitware.com>
CommitDate: Tue Jul 30 13:53:44 2019 -0400
Ninja: Record dyndep support by Ninja 1.10
Upstream Ninja 1.10 and above support the `dyndep` feature we need
for Fortran.
diff --git a/Source/cmGlobalNinjaGenerator.cxx b/Source/cmGlobalNinjaGenerator.cxx
index 4656d03..0b68966 100644
--- a/Source/cmGlobalNinjaGenerator.cxx
+++ b/Source/cmGlobalNinjaGenerator.cxx
@@ -548,7 +548,11 @@ void cmGlobalNinjaGenerator::CheckNinjaFeatures()
this->NinjaSupportsMultilineDepfile = !cmSystemTools::VersionCompare(
cmSystemTools::OP_LESS, this->NinjaVersion.c_str(),
RequiredNinjaVersionForMultilineDepfile().c_str());
- {
+ this->NinjaSupportsDyndeps = !cmSystemTools::VersionCompare(
+ cmSystemTools::OP_LESS, this->NinjaVersion.c_str(),
+ RequiredNinjaVersionForDyndeps().c_str());
+ if (!this->NinjaSupportsDyndeps) {
+ // The ninja version number is not new enough to have upstream support.
// Our ninja branch adds ".dyndep-#" to its version number,
// where '#' is a feature-specific version number. Extract it.
static std::string const k_DYNDEP_ = ".dyndep-";
@@ -586,9 +590,10 @@ bool cmGlobalNinjaGenerator::CheckFortran(cmMakefile* mf) const
"The Ninja generator does not support Fortran using Ninja version\n"
" " + this->NinjaVersion + "\n"
"due to lack of required features. "
- "Kitware has implemented the required features but as of this version "
- "of CMake they have not been integrated to upstream ninja. "
- "Pending integration, Kitware maintains a branch at:\n"
+ "Kitware has implemented the required features and they have been "
+ "merged to upstream ninja for inclusion in Ninja 1.10 and higher. "
+ "As of this version of CMake, Ninja 1.10 has not been released. "
+ "Meanwhile, Kitware maintains a branch of Ninja at:\n"
" https://github.com/Kitware/ninja/tree/features-for-fortran#readme\n"
"with the required features. "
"One may build ninja from that branch to get support for Fortran."
diff --git a/Source/cmGlobalNinjaGenerator.h b/Source/cmGlobalNinjaGenerator.h
index c752bab..db64031 100644
--- a/Source/cmGlobalNinjaGenerator.h
+++ b/Source/cmGlobalNinjaGenerator.h
@@ -322,6 +322,7 @@ public:
{
return "1.9";
}
+ static std::string RequiredNinjaVersionForDyndeps() { return "1.10"; }
bool SupportsConsolePool() const;
bool SupportsImplicitOuts() const;
bool SupportsManifestRestat() const;
https://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=83368b4dd539f287942e26e53ed49d4ec3a18db0
commit 83368b4dd539f287942e26e53ed49d4ec3a18db0
Author: Brad King <brad.king at kitware.com>
AuthorDate: Tue Jul 30 13:53:41 2019 -0400
Commit: Brad King <brad.king at kitware.com>
CommitDate: Tue Jul 30 13:53:41 2019 -0400
Ninja: Drop unused dyndep version check
Our dyndep support version 1 has been merged to upstream Ninja.
We never developed a second dyndep version, so simply drop our
checks for different versions.
diff --git a/Source/cmGlobalNinjaGenerator.cxx b/Source/cmGlobalNinjaGenerator.cxx
index 87c55bf..4656d03 100644
--- a/Source/cmGlobalNinjaGenerator.cxx
+++ b/Source/cmGlobalNinjaGenerator.cxx
@@ -555,7 +555,11 @@ void cmGlobalNinjaGenerator::CheckNinjaFeatures()
std::string::size_type pos = this->NinjaVersion.find(k_DYNDEP_);
if (pos != std::string::npos) {
const char* fv = &this->NinjaVersion[pos + k_DYNDEP_.size()];
- cmSystemTools::StringToULong(fv, &this->NinjaSupportsDyndeps);
+ unsigned long dyndep = 0;
+ cmSystemTools::StringToULong(fv, &dyndep);
+ if (dyndep == 1) {
+ this->NinjaSupportsDyndeps = true;
+ }
}
}
}
@@ -572,37 +576,24 @@ bool cmGlobalNinjaGenerator::CheckLanguages(
bool cmGlobalNinjaGenerator::CheckFortran(cmMakefile* mf) const
{
- if (this->NinjaSupportsDyndeps == 1) {
+ if (this->NinjaSupportsDyndeps) {
return true;
}
std::ostringstream e;
- if (this->NinjaSupportsDyndeps == 0) {
- /* clang-format off */
- e <<
- "The Ninja generator does not support Fortran using Ninja version\n"
- " " + this->NinjaVersion + "\n"
- "due to lack of required features. "
- "Kitware has implemented the required features but as of this version "
- "of CMake they have not been integrated to upstream ninja. "
- "Pending integration, Kitware maintains a branch at:\n"
- " https://github.com/Kitware/ninja/tree/features-for-fortran#readme\n"
- "with the required features. "
- "One may build ninja from that branch to get support for Fortran."
- ;
- /* clang-format on */
- } else {
- /* clang-format off */
- e <<
- "The Ninja generator in this version of CMake does not support Fortran "
- "using Ninja version\n"
- " " + this->NinjaVersion + "\n"
- "because its 'dyndep' feature version is " <<
- this->NinjaSupportsDyndeps << ". "
- "This version of CMake is aware only of 'dyndep' feature version 1."
- ;
- /* clang-format on */
- }
+ /* clang-format off */
+ e <<
+ "The Ninja generator does not support Fortran using Ninja version\n"
+ " " + this->NinjaVersion + "\n"
+ "due to lack of required features. "
+ "Kitware has implemented the required features but as of this version "
+ "of CMake they have not been integrated to upstream ninja. "
+ "Pending integration, Kitware maintains a branch at:\n"
+ " https://github.com/Kitware/ninja/tree/features-for-fortran#readme\n"
+ "with the required features. "
+ "One may build ninja from that branch to get support for Fortran."
+ ;
+ /* clang-format on */
mf->IssueMessage(MessageType::FATAL_ERROR, e.str());
cmSystemTools::SetFatalErrorOccured();
return false;
diff --git a/Source/cmGlobalNinjaGenerator.h b/Source/cmGlobalNinjaGenerator.h
index 9ca7480..c752bab 100644
--- a/Source/cmGlobalNinjaGenerator.h
+++ b/Source/cmGlobalNinjaGenerator.h
@@ -439,7 +439,7 @@ private:
bool NinjaSupportsImplicitOuts = false;
bool NinjaSupportsManifestRestat = false;
bool NinjaSupportsMultilineDepfile = false;
- unsigned long NinjaSupportsDyndeps = 0;
+ bool NinjaSupportsDyndeps = false;
private:
void InitOutputPathPrefix();
https://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=fd58bb83e6a6dcbe27afd2272d1a9c2722de2482
commit fd58bb83e6a6dcbe27afd2272d1a9c2722de2482
Author: Brad King <brad.king at kitware.com>
AuthorDate: Tue Jul 30 13:53:34 2019 -0400
Commit: Brad King <brad.king at kitware.com>
CommitDate: Tue Jul 30 13:53:34 2019 -0400
Ninja: Use in-class initialization of global generator members
diff --git a/Source/cmGlobalNinjaGenerator.cxx b/Source/cmGlobalNinjaGenerator.cxx
index bad715d..87c55bf 100644
--- a/Source/cmGlobalNinjaGenerator.cxx
+++ b/Source/cmGlobalNinjaGenerator.cxx
@@ -415,14 +415,6 @@ void cmGlobalNinjaGenerator::WriteDefault(std::ostream& os,
cmGlobalNinjaGenerator::cmGlobalNinjaGenerator(cmake* cm)
: cmGlobalCommonGenerator(cm)
- , UsingGCCOnWindows(false)
- , ComputingUnknownDependencies(false)
- , PolicyCMP0058(cmPolicies::WARN)
- , NinjaSupportsConsolePool(false)
- , NinjaSupportsImplicitOuts(false)
- , NinjaSupportsManifestRestat(false)
- , NinjaSupportsMultilineDepfile(false)
- , NinjaSupportsDyndeps(0)
{
#ifdef _WIN32
cm->GetState()->SetWindowsShell(true);
diff --git a/Source/cmGlobalNinjaGenerator.h b/Source/cmGlobalNinjaGenerator.h
index 99afc1d..9ca7480 100644
--- a/Source/cmGlobalNinjaGenerator.h
+++ b/Source/cmGlobalNinjaGenerator.h
@@ -402,7 +402,7 @@ private:
/// The set of dependencies to add to the "all" target.
cmNinjaDeps AllDependencies;
- bool UsingGCCOnWindows;
+ bool UsingGCCOnWindows = false;
/// The set of custom commands we have seen.
std::set<cmCustomCommand const*> CustomCommands;
@@ -412,8 +412,8 @@ private:
/// Whether we are collecting known build outputs and needed
/// dependencies to determine unknown dependencies.
- bool ComputingUnknownDependencies;
- cmPolicies::PolicyStatus PolicyCMP0058;
+ bool ComputingUnknownDependencies = false;
+ cmPolicies::PolicyStatus PolicyCMP0058 = cmPolicies::WARN;
/// The combined explicit dependencies of custom build commands
std::set<std::string> CombinedCustomCommandExplicitDependencies;
@@ -435,11 +435,11 @@ private:
std::string NinjaCommand;
std::string NinjaVersion;
- bool NinjaSupportsConsolePool;
- bool NinjaSupportsImplicitOuts;
- bool NinjaSupportsManifestRestat;
- bool NinjaSupportsMultilineDepfile;
- unsigned long NinjaSupportsDyndeps;
+ bool NinjaSupportsConsolePool = false;
+ bool NinjaSupportsImplicitOuts = false;
+ bool NinjaSupportsManifestRestat = false;
+ bool NinjaSupportsMultilineDepfile = false;
+ unsigned long NinjaSupportsDyndeps = 0;
private:
void InitOutputPathPrefix();
https://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=800c3ab2e0b5eed54221ac37c634938ca3d273ab
commit 800c3ab2e0b5eed54221ac37c634938ca3d273ab
Author: Brad King <brad.king at kitware.com>
AuthorDate: Tue Jul 30 12:07:37 2019 -0400
Commit: Brad King <brad.king at kitware.com>
CommitDate: Tue Jul 30 12:09:43 2019 -0400
FindPythonInterp: Document lack of version filtering for plain `python`
Also drop the `CMakeOnly.MajorVersionSelection-PythonInterp_2` test
because some environments now have a plain `python` executable for
Python 3.
Fixes: #19536
diff --git a/Modules/FindPythonInterp.cmake b/Modules/FindPythonInterp.cmake
index da33301..ccc7d5b 100644
--- a/Modules/FindPythonInterp.cmake
+++ b/Modules/FindPythonInterp.cmake
@@ -39,6 +39,15 @@ If calling both ``find_package(PythonInterp)`` and
``find_package(PythonLibs)``, call ``find_package(PythonInterp)`` first to
get the currently active Python version by default with a consistent version
of PYTHON_LIBRARIES.
+
+.. note::
+
+ A call to ``find_package(PythonInterp ${V})`` for python version ``V``
+ may find a ``python`` executable with no version suffix. In this case
+ no attempt is made to avoid python executables from other versions.
+ Use :module:`FindPython3`, :module:`FindPython2` or :module:`FindPython`
+ instead.
+
#]=======================================================================]
unset(_Python_NAMES)
diff --git a/Tests/CMakeOnly/CMakeLists.txt b/Tests/CMakeOnly/CMakeLists.txt
index 1aeab8b..19f3f79 100644
--- a/Tests/CMakeOnly/CMakeLists.txt
+++ b/Tests/CMakeOnly/CMakeLists.txt
@@ -84,5 +84,5 @@ function(add_major_test module)
endfunction()
add_major_test(PythonLibs VERSIONS 2 3 VERSION_VAR PYTHONLIBS_VERSION_STRING)
-add_major_test(PythonInterp NOLANG VERSIONS 2 3 VERSION_VAR PYTHON_VERSION_STRING)
+add_major_test(PythonInterp NOLANG VERSIONS 3 VERSION_VAR PYTHON_VERSION_STRING)
add_major_test(Qt VERSIONS 3 4 VERSION_VAR QT_VERSION_STRING)
https://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=627fc5b44f7c5fab2d6bc81ae3a0d9b06cb9f3bf
commit 627fc5b44f7c5fab2d6bc81ae3a0d9b06cb9f3bf
Author: Tetragramm <gaussgun+github at gmail.com>
AuthorDate: Mon Jul 29 18:17:49 2019 -0500
Commit: Brad King <brad.king at kitware.com>
CommitDate: Tue Jul 30 11:45:33 2019 -0400
ExternalProject: Avoid unnecessary checkout on clone
The clone step checks out the cloned branch but is always followed by an
explicit checkout of the desired `GIT_TAG`. Tell `git clone` not to
check out.
diff --git a/Modules/ExternalProject.cmake b/Modules/ExternalProject.cmake
index 14fc231..e55ed46 100644
--- a/Modules/ExternalProject.cmake
+++ b/Modules/ExternalProject.cmake
@@ -1079,7 +1079,7 @@ function(_ep_write_gitclone_script script_filename source_dir git_EXECUTABLE git
message(FATAL_ERROR "Tag for git checkout should not be empty.")
endif()
- set(git_clone_options)
+ set(git_clone_options "--no-checkout")
if(git_shallow)
if(NOT GIT_VERSION_STRING VERSION_LESS 1.7.10)
list(APPEND git_clone_options "--depth 1 --no-single-branch")
-----------------------------------------------------------------------
Summary of changes:
Modules/ExternalProject.cmake | 2 +-
Modules/FindPythonInterp.cmake | 9 ++++++
Source/cmGlobalNinjaGenerator.cxx | 62 ++++++++++++++++-----------------------
Source/cmGlobalNinjaGenerator.h | 17 ++++++-----
Source/cmSourceGroupCommand.cxx | 15 +++-------
Tests/CMakeOnly/CMakeLists.txt | 2 +-
6 files changed, 49 insertions(+), 58 deletions(-)
hooks/post-receive
--
CMake
More information about the Cmake-commits
mailing list