Revert "ProjectExplorer: Upstream CMake's handling of outdated builds"

This reverts commit 8536504caa.

Break CustomRunConfigurations.

Change-Id: I43020bd34780cb5fd02bbe640a817370d73cca18
Reviewed-by: Alessandro Portale <alessandro.portale@qt.io>

src/plugins/cmakeprojectmanager/cmakerunconfiguration.cpp

@@ -76,6 +76,28 @@ void CMakeRunConfiguration::doAdditionalSetup(const RunConfigurationCreationInfo
     updateTargetInformation();
 }
 
+bool CMakeRunConfiguration::isBuildTargetValid() const
+{
+    return Utils::anyOf(target()->applicationTargets().list, [this](const BuildTargetInfo &bti) {
+        return bti.buildKey == buildKey();
+    });
+}
+
+void CMakeRunConfiguration::updateEnabledState()
+{
+    if (!isBuildTargetValid())
+        setEnabled(false);
+    else
+        RunConfiguration::updateEnabledState();
+}
+
+QString CMakeRunConfiguration::disabledReason() const
+{
+    if (!isBuildTargetValid())
+        return tr("The project no longer builds the target associated with this run configuration.");
+    return RunConfiguration::disabledReason();
+}
+
 void CMakeRunConfiguration::updateTargetInformation()
 {
     BuildTargetInfo bti = buildTargetInfo();

src/plugins/cmakeprojectmanager/cmakerunconfiguration.h

@@ -38,8 +38,13 @@ public:
     CMakeRunConfiguration(ProjectExplorer::Target *target, Core::Id id);
 
 private:
+    QString disabledReason() const override;
+
     void doAdditionalSetup(const ProjectExplorer::RunConfigurationCreationInfo &) override;
+    bool isBuildTargetValid() const;
     void updateTargetInformation();
+
+    void updateEnabledState() final;
 };
 
 class CMakeRunConfigurationFactory : public ProjectExplorer::RunConfigurationFactory

src/plugins/projectexplorer/runconfiguration.cpp

@@ -211,8 +211,6 @@ void RunConfiguration::setEnabled(bool enabled)
 
 QString RunConfiguration::disabledReason() const
 {
-    if (!target()->hasBuildTarget(m_buildKey))
-        return tr("The project no longer builds the target associated with this run configuration.");
     if (target()->project()->isParsing())
         return tr("The Project is currently being parsed.");
     if (!target()->project()->hasParsingData()) {
@@ -247,14 +245,9 @@ QWidget *RunConfiguration::createConfigurationWidget()
 
 void RunConfiguration::updateEnabledState()
 {
-    if (!target()->hasBuildTarget(m_buildKey)) {
+    Project *p = target()->project();
-        // This apparently may happen for cmake builds where also outdated
+
-        // RunConfigurations are kept when builds change.
+    setEnabled(!p->isParsing() && p->hasParsingData());
-        setEnabled(false); // Might happen for CMake.
-    } else {
-        Project *p = target()->project();
-        setEnabled(!p->isParsing() && p->hasParsingData());
-    }
 }
 
 void RunConfiguration::addAspectFactory(const AspectFactory &aspectFactory)

src/plugins/projectexplorer/target.cpp

@@ -365,13 +365,6 @@ BuildTargetInfo Target::buildTarget(const QString &buildKey) const
     });
 }
 
-bool Target::hasBuildTarget(const QString &buildKey) const
-{
-    return Utils::anyOf(d->m_appTargets.list, [buildKey](const BuildTargetInfo &bti) {
-        return bti.buildKey == buildKey;
-    });
-}
-
 QList<ProjectConfiguration *> Target::projectConfigurations() const
 {
     QList<ProjectConfiguration *> result;

src/plugins/projectexplorer/target.h

@@ -85,7 +85,6 @@ public:
     void setApplicationTargets(const BuildTargetInfoList &appTargets);
     BuildTargetInfoList applicationTargets() const;
     BuildTargetInfo buildTarget(const QString &buildKey) const;
-    bool hasBuildTarget(const QString &buildKey) const;
 
     QList<ProjectConfiguration *> projectConfigurations() const;