summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMichal Klocek <michal.klocek@qt.io>2019-09-04 19:32:05 +0200
committerMichal Klocek <michal.klocek@qt.io>2019-09-05 07:48:42 +0200
commit2cc7c21820d9927874849a82cf56b42ceeb56987 (patch)
treee92187e3ff48b4ca80ffad3a7dffd4be255349bc
parent80d79bb694911e358dc7458f942444156be82567 (diff)
Clean up architecture support testwip/qtpdf
Make it a private feature so test is not run twice. Change-Id: I1dcea41edf5dbc6992e76991d92278a5a5c13e55 Reviewed-by: Jörg Bornemann <joerg.bornemann@qt.io>
-rw-r--r--configure.pri13
-rw-r--r--src/buildtools/config/platform.pri14
-rw-r--r--src/buildtools/configure.json9
3 files changed, 16 insertions, 20 deletions
diff --git a/configure.pri b/configure.pri
index acb345ae..b7bb0f34 100644
--- a/configure.pri
+++ b/configure.pri
@@ -104,19 +104,20 @@ defineTest(qtConfTest_detectBison) {
defineTest(qtConfTest_detectPlatform) {
!qtwebengine_isPlatformSupported() {
- qtLog("Platform not supported".)
+ qtLog("Platform not supported.")
return(false)
}
return(true)
}
defineTest(qtConfTest_detectArch) {
- !qtwebengine_isArchSupported() {
- qtLog("Architecture not supported".)
- return(false)
- }
- return(true)
+ contains(QT_ARCH, "i386")|contains(QT_ARCH, "x86_64"): return(true)
+ contains(QT_ARCH, "arm")|contains(QT_ARCH, "arm64"): return(true)
+ contains(QT_ARCH, "mips"): return(true)
+ qtLog("Architecture not supported.")
+ return(false)
}
+
defineTest(qtConfTest_detectFlex) {
flex = $$qtConfFindGnuTool("flex$$EXE_SUFFIX")
isEmpty(flex) {
diff --git a/src/buildtools/config/platform.pri b/src/buildtools/config/platform.pri
index 8ff3da4c..8645e682 100644
--- a/src/buildtools/config/platform.pri
+++ b/src/buildtools/config/platform.pri
@@ -66,16 +66,6 @@ defineTest(qtwebengine_isPlatformSupported) {
return(false)
}
-defineTest(qtwebengine_isArchSupported) {
- contains(QT_ARCH, "i386")|contains(QT_ARCH, "x86_64"): return(true)
- contains(QT_ARCH, "arm")|contains(QT_ARCH, "arm64"): return(true)
- contains(QT_ARCH, "mips"): return(true)
-# contains(QT_ARCH, "mips64"): return(true)
-
- qtwebengine_skipBuild("QtWebEngine can only be built for x86, x86-64, ARM, Aarch64, and MIPSel architectures.")
- return(false)
-}
-
defineTest(qtwebengine_isGCCVersionSupported) {
# Keep in sync with src/webengine/doc/src/qtwebengine-platform-notes.qdoc
greaterThan(QMAKE_GCC_MAJOR_VERSION, 4):return(true)
@@ -190,8 +180,8 @@ defineReplace(qtwebengine_checkError) {
return(false)
}
- !qtwebengine_isArchSupported() {
- isEmpty(skipBuildReason): qtwebengine_skipBuild("Unknown error. Architecture unsupported.")
+ !qtConfig(webengine-arch-support) {
+ qtwebengine_skipBuild("QtWebEngine can only be built for x86, x86-64, ARM, Aarch64, and MIPSel architectures.")
return(false)
}
diff --git a/src/buildtools/configure.json b/src/buildtools/configure.json
index f941f854..3561803e 100644
--- a/src/buildtools/configure.json
+++ b/src/buildtools/configure.json
@@ -357,7 +357,7 @@
"label": "platform supported",
"type": "detectPlatform"
},
- "webengine-architecture": {
+ "webengine-arch-support": {
"label": "architecture supported",
"type": "detectArch"
},
@@ -380,8 +380,8 @@
&& features.webengine-flex
&& features.webengine-submodule
&& features.webengine-nowhitespace
+ && features.webengine-arch-support
&& tests.webengine-platform
- && tests.webengine-architecture
&& !config.static
&& (!config.linux || features.pkg-config)
&& (!features.xcb || features.webengine-ozone-x11)
@@ -663,6 +663,11 @@
"label": "nowhitespace",
"condition" : "tests.webengine-nowhitespace",
"output": [ "privateFeature" ]
+ },
+ "webengine-arch-support": {
+ "label": "architecture",
+ "condition" : "tests.webengine-arch-support",
+ "output": [ "privateFeature" ]
}
},
"report": [