From b39687645df5088a8aa72dab91a7072b19f1fa0c Mon Sep 17 00:00:00 2001
From: Tobias Hunger <tobias.hunger@qt.io>
Date: Mon, 6 Nov 2017 17:15:06 +0100
Subject: [PATCH] TargetSetupPage: Ignore changes while ProjectImporter is
 updating

This causes the same kit to be processed repeatedly, which should not
cause harm, but waste CPU cycles.

Change-Id: Ie42272aacec6d8350398b65973570282c0c8d287
Reviewed-by: Tim Jenssen <tim.jenssen@qt.io>
---
 src/plugins/projectexplorer/targetsetuppage.cpp | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/plugins/projectexplorer/targetsetuppage.cpp b/src/plugins/projectexplorer/targetsetuppage.cpp
index 72e95ca8661..c7c37e5e2d4 100644
--- a/src/plugins/projectexplorer/targetsetuppage.cpp
+++ b/src/plugins/projectexplorer/targetsetuppage.cpp
@@ -344,12 +344,12 @@ void TargetSetupPage::handleKitAddition(Kit *k)
 
 void TargetSetupPage::handleKitRemoval(Kit *k)
 {
-    if (m_importer)
-        m_importer->cleanupKit(k);
-
     if (isUpdating())
         return;
 
+    if (m_importer)
+        m_importer->cleanupKit(k);
+
     removeWidget(k);
     updateVisibility();
 }