Amend "ProjectExplorer: Introduce and use a new MultiSelectionAspect"

This partially reverts commit 34798b3efd,
which breaks binary compatibility.

Change-Id: I17d55760fac599bfba1f5a9726d81898e6979926
Reviewed-by: Eike Ziller <eike.ziller@qt.io>

src/plugins/projectexplorer/makestep.cpp

@@ -417,6 +417,22 @@ QWidget *MakeStep::createConfigWidget()
     return widget;
 }
 
+bool MakeStep::buildsTarget(const QString &target) const
+{
+    return m_buildTargetsAspect->value().contains(target);
+}
+
+void MakeStep::setBuildTarget(const QString &target, bool on)
+{
+    QStringList old = m_buildTargetsAspect->value();
+    if (on && !old.contains(target))
+         old << target;
+    else if (!on && old.contains(target))
+        old.removeOne(target);
+
+    m_buildTargetsAspect->setValue(old);
+}
+
 QStringList MakeStep::availableTargets() const
 {
     return m_buildTargetsAspect->allValues();

src/plugins/projectexplorer/makestep.h

@@ -78,6 +78,11 @@ public:
 
     Utils::Environment makeEnvironment() const;
 
+    // FIXME: All unused, remove in 4.15.
+    void setBuildTarget(const QString &buildTarget) { setSelectedBuildTarget(buildTarget); }
+    bool buildsTarget(const QString &target) const;
+    void setBuildTarget(const QString &target, bool on);
+
 protected:
     void supportDisablingForSubdirs() { m_disablingForSubDirsSupported = true; }
     virtual QStringList displayArguments() const;
@@ -87,6 +92,7 @@ private:
     QStringList jobArguments() const;
 
     Utils::MultiSelectionAspect *m_buildTargetsAspect = nullptr;
+    QStringList m_availableTargets; // FIXME: Unused, remove in 4.15.
     Utils::StringAspect *m_makeCommandAspect = nullptr;
     Utils::StringAspect *m_userArgumentsAspect = nullptr;
     Utils::AspectContainer *m_jobCountContainer = nullptr;