Treat a product file the same way as other build system files...

... when doing change tracking after restoring a build graph.

Changes in one product can influence properties in other products
via the surrounding project.

Change-Id: I4294b326280e676939de9533309fdfa9146f9e92
Reviewed-by: Joerg Bornemann <joerg.bornemann@digia.com>

1 files changed, 1 insertions, 4 deletions

diff --git a/src/lib/buildgraph/buildgraphloader.cpp b/src/lib/buildgraph/buildgraphloader.cpp
index fbaeae2b4..67d645a44 100644
--- a/src/lib/buildgraph/buildgraphloader.cpp
+++ b/src/lib/buildgraph/buildgraphloader.cpp
@@ -173,12 +173,10 @@ void BuildGraphLoader::trackProjectChanges(const SetupProjectParameters &paramet
     // can make the list of source files in a product change without the respective file
     // having been touched. In such a case, the build data for that product will have to be set up
     // anew.
-    bool allProductsPotentiallyChanged = false;
     if (hasBuildSystemFileChanged(buildSystemFiles, buildGraphTimeStamp)
             || hasEnvironmentChanged(restoredProject)
             || hasFileExistsResultChanged(restoredProject)) {
         reResolvingNecessary = true;
-        allProductsPotentiallyChanged = true;
     }
 
    if (!reResolvingNecessary)
@@ -214,8 +212,7 @@ void BuildGraphLoader::trackProjectChanges(const SetupProjectParameters &paramet
         }
     }
 
-    if (allProductsPotentiallyChanged)
-        checkAllProductsForChanges(allRestoredProducts, freshProductsByName, changedProducts);
+    checkAllProductsForChanges(allRestoredProducts, freshProductsByName, changedProducts);
     foreach (const ResolvedProductPtr &product, changedProducts) {
         ResolvedProductPtr freshProduct = freshProductsByName.value(product->name);
         if (!freshProduct)