diff --git a/src/plugins/android/androidqtsupport.h b/src/plugins/android/androidqtsupport.h index 3270e99b270..a8bdcdacf5c 100644 --- a/src/plugins/android/androidqtsupport.h +++ b/src/plugins/android/androidqtsupport.h @@ -81,6 +81,9 @@ public: virtual bool validParse(const ProjectExplorer::Target *target) const = 0; virtual bool extraLibraryEnabled(const ProjectExplorer::Target *target) const = 0; virtual Utils::FileName projectFilePath(const ProjectExplorer::Target *target) const = 0; + + virtual void addFiles(const ProjectExplorer::Target *target, const QString &buildKey, + const QStringList &addedFiles) const = 0; }; } // namespace Android diff --git a/src/plugins/qmakeandroidsupport/createandroidmanifestwizard.cpp b/src/plugins/qmakeandroidsupport/createandroidmanifestwizard.cpp index c49dd76f6c1..8c9e1463210 100644 --- a/src/plugins/qmakeandroidsupport/createandroidmanifestwizard.cpp +++ b/src/plugins/qmakeandroidsupport/createandroidmanifestwizard.cpp @@ -34,28 +34,24 @@ #include #include -#include - -#include - #include #include #include #include +#include +#include #include #include #include #include using namespace Android; +using namespace ProjectExplorer; using namespace QmakeAndroidSupport::Internal; using namespace Utils; -using QmakeProjectManager::QmakeProject; -using QmakeProjectManager::QmakeProFile; - // // NoApplicationProFilePage // @@ -73,10 +69,8 @@ NoApplicationProFilePage::NoApplicationProFilePage(CreateAndroidManifestWizard * // // ChooseProFilePage // -ChooseProFilePage::ChooseProFilePage(CreateAndroidManifestWizard *wizard, - const QList &files, - const QmakeProFile *select) : - m_wizard(wizard) +ChooseProFilePage::ChooseProFilePage(CreateAndroidManifestWizard *wizard) + : m_wizard(wizard) { QFormLayout *fl = new QFormLayout(this); QLabel *label = new QLabel(this); @@ -84,10 +78,17 @@ ChooseProFilePage::ChooseProFilePage(CreateAndroidManifestWizard *wizard, label->setText(tr("Select the .pro file for which you want to create the Android template files.")); fl->addRow(label); + Target *target = wizard->target(); + QString currentBuildTarget; + if (RunConfiguration *rc = target->activeRunConfiguration()) + currentBuildTarget = rc->buildKey(); + m_comboBox = new QComboBox(this); - for (QmakeProFile *file : files) { - m_comboBox->addItem(file->displayName(), QVariant::fromValue(static_cast(file))); // TODO something more? - if (file == select) + const BuildTargetInfoList buildTargets = wizard->target()->applicationTargets(); + for (const BuildTargetInfo &bti : buildTargets.list) { + const QString displayName = bti.buildKey; + m_comboBox->addItem(displayName, QVariant(bti.buildKey)); // TODO something more? + if (bti.buildKey == currentBuildTarget) m_comboBox->setCurrentIndex(m_comboBox->count() - 1); } @@ -102,7 +103,7 @@ ChooseProFilePage::ChooseProFilePage(CreateAndroidManifestWizard *wizard, void ChooseProFilePage::nodeSelected(int index) { Q_UNUSED(index) - m_wizard->setProFile(static_cast(m_comboBox->itemData(m_comboBox->currentIndex()).value())); + m_wizard->setBuildKey(m_comboBox->itemData(m_comboBox->currentIndex()).toString()); } @@ -153,8 +154,11 @@ ChooseDirectoryPage::ChooseDirectoryPage(CreateAndroidManifestWizard *wizard) void ChooseDirectoryPage::checkPackageSourceDir() { - QString projectDir = m_wizard->proFile()->filePath().toFileInfo().absolutePath(); - QString newDir = m_androidPackageSourceDir->path(); + const QString buildKey = m_wizard->buildKey(); + const BuildTargetInfo bti = m_wizard->target()->applicationTargets().buildTargetInfo(buildKey); + const QString projectDir = bti.projectFilePath.toFileInfo().absolutePath(); + + const QString newDir = m_androidPackageSourceDir->path(); bool isComplete = QFileInfo(projectDir) != QFileInfo(newDir); m_sourceDirectoryWarning->setVisible(!isComplete); @@ -173,13 +177,20 @@ bool ChooseDirectoryPage::isComplete() const void ChooseDirectoryPage::initializePage() { - QString androidPackageDir = m_wizard->proFile()->singleVariableValue(QmakeProjectManager::Variable::AndroidPackageSourceDir); + const QString buildKey = m_wizard->buildKey(); + const BuildTargetInfo bti = m_wizard->target()->applicationTargets().buildTargetInfo(buildKey); + const QString projectDir = bti.projectFilePath.toFileInfo().absolutePath(); + + AndroidQtSupport *qtSupport = AndroidManager::androidQtSupport(m_wizard->target()); + const QString androidPackageDir + = qtSupport->targetDataItem(Android::Constants::AndroidPackageSourceDir, m_wizard->target()); + if (androidPackageDir.isEmpty()) { m_label->setText(tr("Select the Android package source directory.\n\n" "The files in the Android package source directory are copied to the build directory's " "Android directory and the default files are overwritten.")); - m_androidPackageSourceDir->setPath(m_wizard->proFile()->filePath().toFileInfo().absolutePath().append(QLatin1String("/android"))); + m_androidPackageSourceDir->setPath(projectDir + "/android"); connect(m_androidPackageSourceDir, &PathChooser::rawPathChanged, this, &ChooseDirectoryPage::checkPackageSourceDir); } else { @@ -196,39 +207,34 @@ void ChooseDirectoryPage::initializePage() // CreateAndroidManifestWizard // CreateAndroidManifestWizard::CreateAndroidManifestWizard(ProjectExplorer::Target *target) - : m_target(target), m_proFile(0), m_copyState(Ask) + : m_target(target), m_copyState(Ask) { setWindowTitle(tr("Create Android Template Files Wizard")); - QmakeProject *project = static_cast(target->project()); - QList files = project->applicationProFiles(); + const BuildTargetInfoList buildTargets = target->applicationTargets(); QtSupport::BaseQtVersion *version = QtSupport::QtKitInformation::qtVersion(target->kit()); m_copyGradle = version && version->qtVersion() >= QtSupport::QtVersionNumber(5, 4, 0); - const QmakeProFile *currentRunNode = nullptr; - if (ProjectExplorer::RunConfiguration *rc = target->activeRunConfiguration()) - currentRunNode = project->rootProFile()->findProFile(FileName::fromString(rc->buildKey())); - - if (files.isEmpty()) { + if (buildTargets.list.isEmpty()) { // oh uhm can't create anything addPage(new NoApplicationProFilePage(this)); - } else if (files.size() == 1) { - setProFile(files.first()); + } else if (buildTargets.list.size() == 1) { + setBuildKey(buildTargets.list.first().buildKey); addPage(new ChooseDirectoryPage(this)); } else { - addPage(new ChooseProFilePage(this, files, currentRunNode)); + addPage(new ChooseProFilePage(this)); addPage(new ChooseDirectoryPage(this)); } } -QmakeProjectManager::QmakeProFile *CreateAndroidManifestWizard::proFile() const +QString CreateAndroidManifestWizard::buildKey() const { - return m_proFile; + return m_buildKey; } -void CreateAndroidManifestWizard::setProFile(QmakeProjectManager::QmakeProFile *node) +void CreateAndroidManifestWizard::setBuildKey(const QString &buildKey) { - m_proFile = node; + m_buildKey = buildKey; } void CreateAndroidManifestWizard::setDirectory(const QString &directory) @@ -337,24 +343,33 @@ void CreateAndroidManifestWizard::createAndroidTemplateFiles() AndroidManager::updateGradleProperties(m_target); } - m_proFile->addFiles(addedFiles); - if (m_proFile->singleVariableValue(QmakeProjectManager::Variable::AndroidPackageSourceDir).isEmpty()) { + AndroidQtSupport *qtSupport = AndroidManager::androidQtSupport(m_target); + qtSupport->addFiles(m_target, m_buildKey, addedFiles); + + const QString androidPackageDir + = qtSupport->targetDataItem(Android::Constants::AndroidPackageSourceDir, m_target); + + if (androidPackageDir.isEmpty()) { // and now time for some magic - QString value = QLatin1String("$$PWD/") - + m_proFile->filePath().toFileInfo().absoluteDir().relativeFilePath(m_directory); - bool result = - m_proFile->setProVariable(QLatin1String("ANDROID_PACKAGE_SOURCE_DIR"), QStringList(value)); + const BuildTargetInfo bti = m_target->applicationTargets().buildTargetInfo(m_buildKey); + const QString value = "$$PWD/" + bti.projectFilePath.toFileInfo().absoluteDir().relativeFilePath(m_directory); + bool result = qtSupport->setTargetData(Android::Constants::AndroidPackageSourceDir, {value}, m_target); if (!result) { QMessageBox::warning(this, tr("Project File not Updated"), - tr("Could not update the .pro file %1.") - .arg(m_proFile->filePath().toUserOutput())); + tr("Could not update the project file %1.") + .arg(bti.projectFilePath.toUserOutput())); } } Core::EditorManager::openEditor(m_directory + QLatin1String("/AndroidManifest.xml")); } +ProjectExplorer::Target *CreateAndroidManifestWizard::target() const +{ + return m_target; +} + void CreateAndroidManifestWizard::accept() { createAndroidTemplateFiles(); diff --git a/src/plugins/qmakeandroidsupport/createandroidmanifestwizard.h b/src/plugins/qmakeandroidsupport/createandroidmanifestwizard.h index cdd49b247f6..3c207d5dbf9 100644 --- a/src/plugins/qmakeandroidsupport/createandroidmanifestwizard.h +++ b/src/plugins/qmakeandroidsupport/createandroidmanifestwizard.h @@ -36,7 +36,6 @@ class QFormLayout; QT_END_NAMESPACE namespace ProjectExplorer { class Target; } -namespace QmakeProjectManager { class QmakeProFile; } namespace QmakeAndroidSupport { namespace Internal { @@ -56,9 +55,8 @@ class ChooseProFilePage : public QWizardPage { Q_OBJECT public: - ChooseProFilePage(CreateAndroidManifestWizard *wizard, - const QList &nodes, - const QmakeProjectManager::QmakeProFile *select); + explicit ChooseProFilePage(CreateAndroidManifestWizard *wizard); + private: void nodeSelected(int index); private: @@ -92,8 +90,8 @@ class CreateAndroidManifestWizard : public Utils::Wizard public: CreateAndroidManifestWizard(ProjectExplorer::Target *target); - QmakeProjectManager::QmakeProFile *proFile() const; - void setProFile(QmakeProjectManager::QmakeProFile *proFile); + QString buildKey() const; + void setBuildKey(const QString &buildKey); void accept(); bool copyGradle(); @@ -101,6 +99,8 @@ public: void setDirectory(const QString &directory); void setCopyGradle(bool copy); + ProjectExplorer::Target *target() const; + private: enum CopyState { Ask, @@ -112,7 +112,7 @@ private: void createAndroidManifestFile(); void createAndroidTemplateFiles(); ProjectExplorer::Target *m_target; - QmakeProjectManager::QmakeProFile *m_proFile; + QString m_buildKey; QString m_directory; CopyState m_copyState; bool m_copyGradle; diff --git a/src/plugins/qmakeandroidsupport/qmakeandroidsupport.cpp b/src/plugins/qmakeandroidsupport/qmakeandroidsupport.cpp index 4a8d09b3834..d18df2b8b27 100644 --- a/src/plugins/qmakeandroidsupport/qmakeandroidsupport.cpp +++ b/src/plugins/qmakeandroidsupport/qmakeandroidsupport.cpp @@ -124,6 +124,8 @@ bool QmakeAndroidSupport::setTargetData(Core::Id role, const QStringList &values QString var; if (role == Android::Constants::AndroidExtraLibs) var = "ANDROID_EXTRA_LIBS"; + else if (role == Android::Constants::AndroidPackageSourceDir) + var = "ANDROID_PACKAGE_SOURCE_DIR"; if (var.isEmpty()) return false; @@ -203,5 +205,15 @@ void QmakeAndroidSupport::manifestSaved(const ProjectExplorer::Target *target) qbc->manifestSaved(); } +void QmakeAndroidSupport::addFiles(const ProjectExplorer::Target *target, + const QString &buildKey, + const QStringList &addedFiles) const +{ + QmakeProject *project = static_cast(target->project()); + QmakeProFile *currentRunNode = project->rootProFile()->findProFile(FileName::fromString(buildKey)); + QTC_ASSERT(currentRunNode, return); + currentRunNode->addFiles(addedFiles); +} + } // namespace Internal } // namespace QmakeAndroidSupport diff --git a/src/plugins/qmakeandroidsupport/qmakeandroidsupport.h b/src/plugins/qmakeandroidsupport/qmakeandroidsupport.h index ffaf8208529..a8a4920ded5 100644 --- a/src/plugins/qmakeandroidsupport/qmakeandroidsupport.h +++ b/src/plugins/qmakeandroidsupport/qmakeandroidsupport.h @@ -49,6 +49,9 @@ public: bool validParse(const ProjectExplorer::Target *target) const override; bool extraLibraryEnabled(const ProjectExplorer::Target *target) const override; Utils::FileName projectFilePath(const ProjectExplorer::Target *target) const override; + + void addFiles(const ProjectExplorer::Target *target, const QString &buildKey, + const QStringList &addedFiles) const override; }; } // namespace Internal