From 833b86d5e2edf67005ccbe0cf4b0fb0eeb7d4fcb Mon Sep 17 00:00:00 2001
From: dt <qtc-committer@nokia.com>
Date: Tue, 27 Oct 2009 16:13:09 +0100
Subject: [PATCH] Fix that the cmake plugin did not allow removing build
 targets

---
 src/plugins/cmakeprojectmanager/makestep.cpp | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/plugins/cmakeprojectmanager/makestep.cpp b/src/plugins/cmakeprojectmanager/makestep.cpp
index 1d88b4cddac..3a9b975882e 100644
--- a/src/plugins/cmakeprojectmanager/makestep.cpp
+++ b/src/plugins/cmakeprojectmanager/makestep.cpp
@@ -132,9 +132,11 @@ void MakeStep::setBuildTarget(const QString &buildConfiguration, const QString &
 {
     QStringList old = value(buildConfiguration, "buildTargets").toStringList();
     if (on && !old.contains(target))
-        setValue(buildConfiguration, "buildTargets", old << target);
+        old.append(target);
     else if(!on && old.contains(target))
-        setValue(buildConfiguration, "buildTargets", old.removeOne(target));
+        old.removeOne(target);
+
+    setValue(buildConfiguration, "buildTargets", old);
 }
 
 QStringList MakeStep::additionalArguments(const QString &buildConfiguration) const
-- 
GitLab