diff options
author | Tobias Hunger <tobias.hunger@qt.io> | 2019-09-30 14:34:31 +0200 |
---|---|---|
committer | Tobias Hunger <tobias.hunger@qt.io> | 2019-10-01 11:46:43 +0000 |
commit | b489bfd6155a150c337085ad4ac040973391ac8e (patch) | |
tree | 94a1c079d01c6aaa087c16316ec45543d4bc53d6 /src/plugins/android | |
parent | fc2e22dad247445ed4134afbbaff7d9960fd18e2 (diff) | |
download | qt-creator-b489bfd6155a150c337085ad4ac040973391ac8e.tar.gz |
BaseQtVersion: remove qmakeProperty(...) getter
Qt 6 will not use qmake to identify a Qt version, so this can not
be part of the public interface of BaseQtVersion anymore.
Provide getters for the information actually read via qmakeProperty(...).
Use the getters whenever possible.
Change-Id: Iadbee80b75e4f8b06caf90e7ed69fae2029b4dd7
Reviewed-by: hjk <hjk@qt.io>
Diffstat (limited to 'src/plugins/android')
-rw-r--r-- | src/plugins/android/androidbuildapkstep.cpp | 2 | ||||
-rw-r--r-- | src/plugins/android/androiddeployqtstep.cpp | 2 | ||||
-rw-r--r-- | src/plugins/android/androidmanager.cpp | 11 | ||||
-rw-r--r-- | src/plugins/android/createandroidmanifestwizard.cpp | 8 |
4 files changed, 10 insertions, 13 deletions
diff --git a/src/plugins/android/androidbuildapkstep.cpp b/src/plugins/android/androidbuildapkstep.cpp index 4cff7c90ab..bf89bd4a93 100644 --- a/src/plugins/android/androidbuildapkstep.cpp +++ b/src/plugins/android/androidbuildapkstep.cpp @@ -212,7 +212,7 @@ bool AndroidBuildApkStep::init() if (!AbstractProcessStep::init()) return false; - QString command = version->qmakeProperty("QT_HOST_BINS"); + QString command = version->hostBinPath().toString(); if (!command.endsWith('/')) command += '/'; command += "androiddeployqt"; diff --git a/src/plugins/android/androiddeployqtstep.cpp b/src/plugins/android/androiddeployqtstep.cpp index ab9a0745a6..fe6e2f30b5 100644 --- a/src/plugins/android/androiddeployqtstep.cpp +++ b/src/plugins/android/androiddeployqtstep.cpp @@ -240,7 +240,7 @@ bool AndroidDeployQtStep::init() emit addOutput(tr("Cannot find the androiddeploy Json file."), OutputFormat::Stderr); return false; } - m_command = FilePath::fromString(version->qmakeProperty("QT_HOST_BINS")); + m_command = version->hostBinPath(); if (m_command.isEmpty()) { emit addOutput(tr("Cannot find the androiddeployqt tool."), OutputFormat::Stderr); return false; diff --git a/src/plugins/android/androidmanager.cpp b/src/plugins/android/androidmanager.cpp index b0f887a74d..4958bd02a3 100644 --- a/src/plugins/android/androidmanager.cpp +++ b/src/plugins/android/androidmanager.cpp @@ -238,9 +238,8 @@ int AndroidManager::minimumSDK(const ProjectExplorer::Kit *kit) int minSDKVersion = -1; QtSupport::BaseQtVersion *version = QtSupport::QtKitAspect::qtVersion(kit); if (version && version->targetDeviceTypes().contains(Constants::ANDROID_DEVICE_TYPE)) { - Utils::FilePath stockManifestFilePath = - Utils::FilePath::fromUserInput(version->qmakeProperty("QT_INSTALL_PREFIX") + - QLatin1String("/src/android/templates/AndroidManifest.xml")); + Utils::FilePath stockManifestFilePath = Utils::FilePath::fromUserInput( + version->prefix().toString() + "/src/android/templates/AndroidManifest.xml"); QDomDocument doc; if (openXmlFile(doc, stockManifestFilePath)) { minSDKVersion = parseMinSdk(doc.documentElement()); @@ -276,7 +275,7 @@ QJsonObject AndroidManager::deploymentSettings(const Target *target) return {}; QJsonObject settings; settings["_description"] = "This file is generated by QtCreator to be read by androiddeployqt and should not be modified by hand."; - settings["qt"] = qt->qmakeProperty("QT_INSTALL_PREFIX"); + settings["qt"] = qt->prefix().toString(); settings["ndk"] = AndroidConfigurations::currentConfig().ndkLocation().toString(); settings["sdk"] = AndroidConfigurations::currentConfig().sdkLocation().toString(); settings["stdcpp-path"] = AndroidConfigurations::currentConfig().toolchainPath().pathAppended("sysroot/usr/lib/").toString(); @@ -728,8 +727,8 @@ bool AndroidManager::updateGradleProperties(ProjectExplorer::Target *target, con const QString gradlePropertiesPath = packageSourceDir.pathAppended("gradle.properties").toString(); GradleProperties gradleProperties = readGradleProperties(gradlePropertiesPath); - gradleProperties["qt5AndroidDir"] = version->qmakeProperty("QT_INSTALL_PREFIX") - .append(QLatin1String("/src/android/java")).toLocal8Bit(); + gradleProperties["qt5AndroidDir"] = (version->prefix().toString() + "/src/android/java") + .toLocal8Bit(); gradleProperties["buildDir"] = ".build"; gradleProperties["androidCompileSdkVersion"] = buildTargetSDK(target).split(QLatin1Char('-')).last().toLocal8Bit(); if (gradleProperties["androidBuildToolsVersion"].isEmpty()) { diff --git a/src/plugins/android/createandroidmanifestwizard.cpp b/src/plugins/android/createandroidmanifestwizard.cpp index 70f75fed98..bac135d7d2 100644 --- a/src/plugins/android/createandroidmanifestwizard.cpp +++ b/src/plugins/android/createandroidmanifestwizard.cpp @@ -322,21 +322,19 @@ void CreateAndroidManifestWizard::createAndroidTemplateFiles() if (!version) return; if (version->qtVersion() < QtSupport::QtVersionNumber(5, 4, 0)) { - const QString src(version->qmakeProperty("QT_INSTALL_PREFIX") - .append(QLatin1String("/src/android/java/AndroidManifest.xml"))); + const QString src = version->prefix().toString() + "/src/android/java/AndroidManifest.xml"; FileUtils::copyRecursively(FilePath::fromString(src), FilePath::fromString(m_directory + QLatin1String("/AndroidManifest.xml")), nullptr, [this, &addedFiles](QFileInfo src, QFileInfo dst, QString *){return copy(src, dst, &addedFiles);}); } else { - const QString src(version->qmakeProperty("QT_INSTALL_PREFIX") - .append(QLatin1String("/src/android/templates"))); + const QString src = version->prefix().toString() + "/src/android/templates"; FileUtils::copyRecursively(FilePath::fromString(src), FilePath::fromString(m_directory), nullptr, [this, &addedFiles](QFileInfo src, QFileInfo dst, QString *){return copy(src, dst, &addedFiles);}); if (m_copyGradle) { - FilePath gradlePath = FilePath::fromString(version->qmakeProperty("QT_INSTALL_PREFIX").append(QLatin1String("/src/3rdparty/gradle"))); + FilePath gradlePath = version->prefix().pathAppended("src/3rdparty/gradle"); if (!gradlePath.exists()) gradlePath = AndroidConfigurations::currentConfig().sdkLocation().pathAppended("/tools/templates/gradle/wrapper"); FileUtils::copyRecursively(gradlePath, FilePath::fromString(m_directory), |