aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAlessandro Portale <alessandro.portale@qt.io>2019-09-28 22:24:13 +0200
committerAlessandro Portale <alessandro.portale@qt.io>2019-09-30 08:05:08 +0000
commit6d974b116ddcc9705266f95d1991a9b51035f61d (patch)
treeff15221efce483b545f3a6d70112843a6b84c84d
parent5f835fd4745c16299b7d2230d110bfb6a47e7095 (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--.gitmodules5
m---------3rdparty/extra-cmake-modules0
m---------3rdparty/kuserfeedback0
-rw-r--r--usagestatistic.pro86
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