fix calculation of added/removed artifacts

We kept lists of added and removed artifacts in ProductBuildData. It was
never quite clear when to invalidate those lists, which led to QBS-635.
Instead we let the RuleNode decide which artifacts it considers as
"added or removed for this rule".

Task-number: QBS-635
Change-Id: I390e0ab775c695045c6e91ade3ac7326692cb314
Reviewed-by: Christian Kandeler <christian.kandeler@digia.com>

1 files changed, 2 insertions, 4 deletions

diff --git a/src/lib/corelib/api/project.cpp b/src/lib/corelib/api/project.cpp
index b8706a8d6..69cb2034a 100644
--- a/src/lib/corelib/api/project.cpp
+++ b/src/lib/corelib/api/project.cpp
@@ -398,10 +398,8 @@ void ProjectPrivate::addFiles(const ProductData &product, const GroupData &group
             resolvedGroup->files << artifact;
         }
         if (resolvedProduct->enabled) {
-            foreach (const SourceArtifactConstPtr &sa, addedSourceArtifacts) {
-                Artifact * const artifact = createArtifact(resolvedProduct, sa, logger);
-                resolvedProduct->registerAddedArtifact(artifact);
-            }
+            foreach (const SourceArtifactConstPtr &sa, addedSourceArtifacts)
+                createArtifact(resolvedProduct, sa, logger);
         }
     }
     doSanityChecks(internalProject, logger);