diff options
author | Alessandro Portale <alessandro.portale@qt.io> | 2019-09-28 22:24:13 +0200 |
---|---|---|
committer | Alessandro Portale <alessandro.portale@qt.io> | 2019-09-30 08:05:08 +0000 |
commit | 6d974b116ddcc9705266f95d1991a9b51035f61d (patch) | |
tree | ff15221efce483b545f3a6d70112843a6b84c84d | |
parent | 5f835fd4745c16299b7d2230d110bfb6a47e7095 (diff) |
Switch to qmake-based KUserFeedback build
...instead of trying to combine a qmake and a CMake build system.
Temporary hack until Qt Creator packages get built with CMake.
Task-number: QTCREATORBUG-22860
Change-Id: Ibc51a26b73c78e6db8cffc8767fe24653aa5391c
Reviewed-by: Eike Ziller <eike.ziller@qt.io>
Reviewed-by: Vitaly Fanaskov <vitaly.fanaskov@qt.io>
Reviewed-by: Tuukka Turunen <tuukka.turunen@qt.io>
Reviewed-by: Alessandro Portale <alessandro.portale@qt.io>
-rw-r--r-- | .gitmodules | 5 | ||||
m--------- | 3rdparty/extra-cmake-modules | 0 | ||||
m--------- | 3rdparty/kuserfeedback | 0 | ||||
-rw-r--r-- | usagestatistic.pro | 86 |
4 files changed, 4 insertions, 87 deletions
diff --git a/.gitmodules b/.gitmodules index 5815ead..0a464e5 100644 --- a/.gitmodules +++ b/.gitmodules @@ -1,6 +1,3 @@ [submodule "3rdparty/kuserfeedback"] path = 3rdparty/kuserfeedback - url = https://git.qt.io/telemetry/kuserfeedback.git -[submodule "3rdparty/extra-cmake-modules"] - path = 3rdparty/extra-cmake-modules - url = https://github.com/KDE/extra-cmake-modules.git + url = https://git.qt.io/aportale/kuserfeedback.git diff --git a/3rdparty/extra-cmake-modules b/3rdparty/extra-cmake-modules deleted file mode 160000 -Subproject 6faff48245c213a472da2e28b2c9ece31177c68 diff --git a/3rdparty/kuserfeedback b/3rdparty/kuserfeedback -Subproject 463cd3dadde31cb007d3385df5b1e2d76fb3dc9 +Subproject 7963324f3d269befa1574c16b13944ba99114d2 diff --git a/usagestatistic.pro b/usagestatistic.pro index e027b41..f9fd69d 100644 --- a/usagestatistic.pro +++ b/usagestatistic.pro @@ -1,8 +1,6 @@ DEFINES += USAGESTATISTIC_LIBRARY -KUSERFEEDBACK_INSTALL_PATH = "$${OUT_PWD}/kuserfeedback" - -INCLUDEPATH *= "$$shell_path($${KUSERFEEDBACK_INSTALL_PATH}/include)" "$${PWD}" +INCLUDEPATH *= "$${PWD}" CONFIG += c++1z QMAKE_CXXFLAGS *= -Wall @@ -89,89 +87,11 @@ QTC_PLUGIN_RECOMMENDS += \ ###### End _dependencies.pri contents ###### -!build_pass|win32 { - EXTRA_CMAKE_MODULES_BUILD_PATH = "$${OUT_PWD}/extra-cmake-modules/build" - EXTRA_CMAKE_MODULES_SOURCE_PATH = "$${PWD}/3rdparty/extra-cmake-modules" - EXTRA_CMAKE_MODULES_INSTALL_PATH = "$${OUT_PWD}/extra-cmake-modules" - - # Configure extra-cmake-modules - - # Create build directory, in case if CMake won't be able do that - system("$$sprintf($$QMAKE_MKDIR_CMD, $$shell_path($${EXTRA_CMAKE_MODULES_BUILD_PATH}))") - - EXTRA_CMAKE_MODULES_CMAKE_CMD = \ - "$$QMAKE_CD $$system_quote($$shell_path($${EXTRA_CMAKE_MODULES_BUILD_PATH})) && \ - cmake -DCMAKE_INSTALL_PREFIX:PATH=$$system_quote($$shell_path($${EXTRA_CMAKE_MODULES_INSTALL_PATH})) \ - $$system_quote($$shell_path($${EXTRA_CMAKE_MODULES_SOURCE_PATH}))" - message("$${EXTRA_CMAKE_MODULES_CMAKE_CMD}") - system("$${EXTRA_CMAKE_MODULES_CMAKE_CMD}") - - # "Build" extra-cmake-modules first time. This step is required to configure KUserFeedback - EXTRA_CMAKE_MODULES_BUILD_CMD = "cmake --build $$system_quote($$shell_path($${EXTRA_CMAKE_MODULES_BUILD_PATH})) --target install" - message("$${EXTRA_CMAKE_MODULES_BUILD_CMD}") - system("$${EXTRA_CMAKE_MODULES_BUILD_CMD}") - - # Configure KUserFeedback - KUSERFEEDBACK_BUILD_PATH = "$${OUT_PWD}/kuserfeedback/build" - KUSERFEEDBACK_SOURCE_PATH = "$${PWD}/3rdparty/kuserfeedback" - - CMAKE_PREFIX_PATHS = "$$shell_path($$[QT_INSTALL_LIBS]/cmake);$$shell_path($${EXTRA_CMAKE_MODULES_INSTALL_PATH}/share/ECM/cmake)" - - BUILD_TYPE = Debug - CONFIG(release, debug|release): BUILD_TYPE = Release - - KUSERFEEDBACK_DEFINES = \ - -DCMAKE_BUILD_TYPE=$${BUILD_TYPE} - - macos: KUSERFEEDBACK_DEFINES *= -DCMAKE_OSX_DEPLOYMENT_TARGET=$${QMAKE_MACOSX_DEPLOYMENT_TARGET} - - KUSERFEEDBACK_COMPONENTS = \ - -DBUILD_SHARED_LIBS=OFF \ - -DENABLE_SURVEY_TARGET_EXPRESSIONS=OFF \ - -DENABLE_PHP=OFF \ - -DENABLE_PHP_UNIT=OFF \ - -DENABLE_TESTING=OFF \ - -DENABLE_DOCS=OFF \ - -DENABLE_CONSOLE=OFF \ - -DENABLE_CLI=OFF \ - -DBUILD_SHARED_LIBS=OFF - - # Create build directory, in case if CMake won't be able do that - system("$$sprintf($$QMAKE_MKDIR_CMD, $$shell_path($${KUSERFEEDBACK_BUILD_PATH}))") - - KUSERFEEDBACK_CMAKE_CMD = \ - "$$QMAKE_CD $$system_quote($$shell_path($${KUSERFEEDBACK_BUILD_PATH})) && \ - cmake $${KUSERFEEDBACK_COMPONENTS} \ - $${KUSERFEEDBACK_DEFINES} \ - -DCMAKE_INSTALL_PREFIX:PATH=$$system_quote($$shell_path($${KUSERFEEDBACK_INSTALL_PATH})) \ - -DCMAKE_PREFIX_PATH=$$system_quote($${CMAKE_PREFIX_PATHS}) \ - -DKDE_INSTALL_LIBDIR=lib \ - $$system_quote($$shell_path($${KUSERFEEDBACK_SOURCE_PATH}))" - message("$${KUSERFEEDBACK_CMAKE_CMD}") - system("$${KUSERFEEDBACK_CMAKE_CMD}") - - buildextracmakemodules.commands = "$${EXTRA_CMAKE_MODULES_BUILD_CMD}" - - buildkuserfeedback.commands = "cmake --build $$system_quote($$shell_path($${KUSERFEEDBACK_BUILD_PATH})) --target install" - buildkuserfeedback.depends = buildextracmakemodules - - # Force build order. Without this flag Make tries building targets - # in a random order when -jN specified. - # All targets themselves (but not the plugin itself) will be built in parallel - notParallel.target = .NOTPARALLEL - - QMAKE_EXTRA_TARGETS += buildextracmakemodules buildkuserfeedback notParallel - PRE_TARGETDEPS += buildkuserfeedback -} +# KUserFeedback +include(3rdparty/kuserfeedback/kuserfeedback.pri) include($$IDE_SOURCE_TREE/src/qtcreatorplugin.pri) -# Put it here to use qtLibraryTargetName function without extra hacks -LIBS *= -L"$$shell_path($${KUSERFEEDBACK_INSTALL_PATH}/lib)" \ - -l$$qtLibraryTargetName(KUserFeedbackCore) \ - -l$$qtLibraryTargetName(KUserFeedbackWidgets) \ - -l$$qtLibraryTargetName(KUserFeedbackCommon) - FORMS += \ ui/usagestatisticwidget.ui \ ui/encouragementwidget.ui |