#1281

Author: jantje

io.sloeber.core/src/io/sloeber/core/api/SloeberProject.java

@@ -406,10 +406,11 @@ public boolean configure(ICProjectDescription prjCDesc, boolean prjDescWritable)
             }
             if (prjDescWritable) {
                 if (myNeedsSyncWithCDT) {
-                    saveProjDesc = syncWithCDT(prjCDesc, prjDescWritable);
+                    saveProjDesc = saveProjDesc || syncWithCDT(prjCDesc, prjDescWritable);
                 }
                 if (myNeedsClean) {
-                    cleanOldData(prjCDesc);
+                    myNeedsClean = cleanOldData(prjCDesc);
+                    saveProjDesc = saveProjDesc || myNeedsClean;
                 }
             }
             return saveProjDesc;
@@ -425,7 +426,8 @@ public boolean configure(ICProjectDescription prjCDesc, boolean prjDescWritable)
             if (myNeedsClean) {
                 // we migrated from a previous sloeber configuration
                 // and we can safely delete the old data
-                cleanOldData(prjCDesc);
+                myNeedsClean = cleanOldData(prjCDesc);
+                saveProjDesc = saveProjDesc || myNeedsClean;
             }
             if (myNeedsSyncWithCDT) {
                 saveProjDesc = saveProjDesc || syncWithCDT(prjCDesc, prjDescWritable);
@@ -451,21 +453,24 @@ private boolean syncWithCDT(ICProjectDescription prjCDesc, boolean prjDescWritab
      * @param prjCDesc
      * @return
      */
-    private void cleanOldData(ICProjectDescription prjCDesc) {
+    private boolean cleanOldData(ICProjectDescription prjCDesc) {
+        boolean needsClean = false;
         IEnvironmentVariableManager envManager = CCorePlugin.getDefault().getBuildEnvironmentManager();
         IContributedEnvironment contribEnv = envManager.getContributedEnvironment();
         for (ICConfigurationDescription confDesc : prjCDesc.getConfigurations()) {
             IEnvironmentVariable[] CurVariables = contribEnv.getVariables(confDesc);
             for (int i = (CurVariables.length - 1); i > 0; i--) {
                 if (CurVariables[i].getName().startsWith("A.")) { //$NON-NLS-1$
                     contribEnv.removeVariable(CurVariables[i].getName(), confDesc);
+                    needsClean = true;
                 }
                 if (CurVariables[i].getName().startsWith("JANTJE.")) { //$NON-NLS-1$
                     contribEnv.removeVariable(CurVariables[i].getName(), confDesc);
+                    needsClean = true;
                 }
             }
         }
-        myNeedsClean = false;
+        return needsClean;
     }
 
     /**