diff options
authorAlexandru Croitor <>2019-11-04 11:10:20 +0100
committerAlexandru Croitor <>2019-11-08 15:10:05 +0000
commitc1ba9bb72a3eeb0bd707e609da156798c51c17f0 (patch)
parentc51fe841f472edb4af5d56d436fa6df201b2fc9a (diff)
Fix PlatformHeaders to be a HEADER_MODULE
The module does not have an associated library, but for some reason on Windows install(EXPORT) still created Targets files which expected a library to be present. If then find_package(Qt6PlatformHeaders) was called, it would fail configuration with not finding the library on disk. Switch the module to be a HEADER_MODULE. Also remove the extend_target call, because there apparently is no need to specify the headers one by one (and also because adding sources to an interface only library would break the configuration step). syncqt will be executed on all the headers under the module source location, regardless of specifying them in the project. Change-Id: Iec4cc8fab8906e6288dae423c878e260dd922ce5 Reviewed-by: Simon Hausmann <> Reviewed-by: Qt CMake Build Bot
1 files changed, 2 insertions, 4 deletions
diff --git a/src/platformheaders/CMakeLists.txt b/src/platformheaders/CMakeLists.txt
index 89daf13410..bbbd1b76b5 100644
--- a/src/platformheaders/CMakeLists.txt
+++ b/src/platformheaders/CMakeLists.txt
@@ -2,11 +2,9 @@
# special case begin
-# FIXME Add the rest of the stuff
+# FIXME Add the rest of the stuff? I don't think any other stuff is needed?
-extend_target(PlatformHeaders CONDITION APPLE SOURCES cocoafunctions/qcocoawindowfunctions.h)
+add_qt_module(PlatformHeaders HEADER_MODULE)