diff options
| author | Christian Kandeler <christian.kandeler@digia.com> | 2013-08-22 16:10:26 +0200 |
|---|---|---|
| committer | Christian Kandeler <christian.kandeler@digia.com> | 2013-08-23 16:39:47 +0200 |
| commit | 6798f6a8709fe82a1c67b656d4321e75936549a2 (patch) | |
| tree | 5dcfb673c0a4850afd4ce002504bcc2bc96f4aa9 /src/lib/buildgraph/rulesapplicator.cpp | |
| parent | fd2a5d4ba2c452d312094cfc215ee9234b0db5bf (diff) | |
| download | qbs-6798f6a8709fe82a1c67b656d4321e75936549a2.tar.gz | |
Fix change tracking for properties requested from prepare scripts.
When evaluating prepare scripts, we currently gather values requested
from products as well as artifacts, but we do not differentiate between
the two cases and upon restoring, we always compare the old property
values to the product properties. This results in an insane amount of
recompiling if any build system file changes due to false positives.
With this patch, we record whether a property was requested from a
product or an artifact, and use the right set of properties when
tracking changes.
Change-Id: Ib1fa4fad41019cfa7d3a10e0a91e7709c2f56414
Reviewed-by: Joerg Bornemann <joerg.bornemann@digia.com>
Diffstat (limited to 'src/lib/buildgraph/rulesapplicator.cpp')
| -rw-r--r-- | src/lib/buildgraph/rulesapplicator.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/lib/buildgraph/rulesapplicator.cpp b/src/lib/buildgraph/rulesapplicator.cpp index 5d3db92cb..d057c3cfb 100644 --- a/src/lib/buildgraph/rulesapplicator.cpp +++ b/src/lib/buildgraph/rulesapplicator.cpp @@ -322,7 +322,7 @@ void RulesApplicator::onPropertyRead(const QScriptValue &object, const QString & const QScriptValue &value) { if (object.objectId() == m_productObjectId) - engine()->addProperty( + engine()->addPropertyRequestedFromProduct( Property(QString(), name, value.toVariant(), Property::PropertyInProduct)); } |