diff options
author | Christian Kandeler <christian.kandeler@qt.io> | 2018-01-29 14:04:56 +0100 |
---|---|---|
committer | Christian Kandeler <christian.kandeler@qt.io> | 2018-02-16 09:33:47 +0000 |
commit | 24471740f276fc7fb3b662bd18ae637127368a9f (patch) | |
tree | 18a5b7657db9cd6b4edbf62da7230434b946fa61 /src/lib/corelib/tools/persistence.cpp | |
parent | fc321f1cca3b9bbcbbf0200a16aff85a85a9a148 (diff) | |
download | qbs-24471740f276fc7fb3b662bd18ae637127368a9f.tar.gz |
Move transformer change tracking out of the build graph loader
The checks that determine whether to re-run prepare scripts and/or
commands due to property or environment changes do not belong into the
build graph loader. Instead, we now do them on demand during the build
process. Advantages:
- The code is at its "natural" place, making it easier to understand
and less fragile.
- There are a lot fewer unnecessary build data invalidations, speeding
up both re-resolving and rebuilding in case only some rules/commands
have changed.
- Re-running commands due to property or environment changes no longer
implies re-running prepare scripts.
- We now catch property changes on generated artifacts, which was not
possible before, because the build graph loader only had access to
the source artifacts of the re-resolved project.
Change-Id: I36b022cf631fa9e8293feec4d6f416c2686539c1
Reviewed-by: Joerg Bornemann <joerg.bornemann@qt.io>
Diffstat (limited to 'src/lib/corelib/tools/persistence.cpp')
-rw-r--r-- | src/lib/corelib/tools/persistence.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/lib/corelib/tools/persistence.cpp b/src/lib/corelib/tools/persistence.cpp index 88f6555f8..8ec6c03f1 100644 --- a/src/lib/corelib/tools/persistence.cpp +++ b/src/lib/corelib/tools/persistence.cpp @@ -49,7 +49,7 @@ namespace qbs { namespace Internal { -static const char QBS_PERSISTENCE_MAGIC[] = "QBSPERSISTENCE_112"; +static const char QBS_PERSISTENCE_MAGIC[] = "QBSPERSISTENCE-113"; NoBuildGraphError::NoBuildGraphError(const QString &filePath) : ErrorInfo(Tr::tr("Build graph not found for configuration '%1'. Expected location was '%2'.") |