summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBrad King <brad.king@kitware.com>2018-12-07 08:04:55 -0500
committerBrad King <brad.king@kitware.com>2018-12-07 08:04:55 -0500
commit3b4e86f26dddbdfd2cd172274ae7c1340f1eba29 (patch)
tree4056647b098fbee735723bc5fa070f8b4f1f6137
parent0becea2d6061e8fa49c5f6a00757efacdd93421a (diff)
parent187481d4d988f45e05f2f7a6e89df89cd64ecfe9 (diff)
downloadcmake-3b4e86f26dddbdfd2cd172274ae7c1340f1eba29.tar.gz
Merge branch 'backport-autogen_revert_implicit_includes' into release-3.13
Merge-request: !2709
-rw-r--r--Help/release/3.13.rst13
-rw-r--r--Source/cmQtAutoGenInitializer.cxx10
-rw-r--r--Tests/QtAutogen/CommonTests.cmake7
3 files changed, 24 insertions, 6 deletions
diff --git a/Help/release/3.13.rst b/Help/release/3.13.rst
index 9f7e61ffd1..68e05c37f0 100644
--- a/Help/release/3.13.rst
+++ b/Help/release/3.13.rst
@@ -239,3 +239,16 @@ Other Changes
These internal implementation modules are also no longer available
to scripts that may have been incorrectly including them, because
they should never have been available in the first place.
+
+Updates
+=======
+
+Changes made since CMake 3.13.0 include the following.
+
+3.13.2
+------
+
+* CMake 3.13.0 included a change to pass compiler implicit include
+ directories to the ``moc`` tool for :prop_tgt:`AUTOMOC`. This has
+ been reverted due to regressing existing builds and will need
+ further investigation before being re-introduced in a later release.
diff --git a/Source/cmQtAutoGenInitializer.cxx b/Source/cmQtAutoGenInitializer.cxx
index ba7a565cae..f9c8c7f2fb 100644
--- a/Source/cmQtAutoGenInitializer.cxx
+++ b/Source/cmQtAutoGenInitializer.cxx
@@ -386,9 +386,13 @@ bool cmQtAutoGenInitializer::InitMoc()
// Moc includes
{
- bool const appendImplicit = (this->QtVersion.Major == 5);
- auto GetIncludeDirs =
- [this, localGen, appendImplicit](std::string const& cfg) -> std::string {
+ // We need to disable this until we have all implicit includes available.
+ // See issue #18669.
+ // bool const appendImplicit = (this->QtVersion.Major == 5);
+
+ auto GetIncludeDirs = [this,
+ localGen](std::string const& cfg) -> std::string {
+ bool const appendImplicit = false;
// Get the include dirs for this target, without stripping the implicit
// include dirs off, see
// https://gitlab.kitware.com/cmake/cmake/issues/13667
diff --git a/Tests/QtAutogen/CommonTests.cmake b/Tests/QtAutogen/CommonTests.cmake
index 01ed7e93c5..ef11ddf282 100644
--- a/Tests/QtAutogen/CommonTests.cmake
+++ b/Tests/QtAutogen/CommonTests.cmake
@@ -23,9 +23,10 @@ endif()
if(QT_TEST_ALLOW_QT_MACROS)
ADD_AUTOGEN_TEST(MocCMP0071)
endif()
-if(QT_TEST_VERSION GREATER 4)
- ADD_AUTOGEN_TEST(MocOsMacros)
-endif()
+# Disabled for issue #18669
+#if(QT_TEST_VERSION GREATER 4)
+# ADD_AUTOGEN_TEST(MocOsMacros)
+#endif()
ADD_AUTOGEN_TEST(UicInclude uicInclude)
ADD_AUTOGEN_TEST(UicInterface QtAutoUicInterface)
ADD_AUTOGEN_TEST(ObjectLibrary someProgram)