diff options
| author | Tristan van Berkom <tristan.vanberkom@codethink.co.uk> | 2020-08-20 14:35:57 +0900 |
|---|---|---|
| committer | Tristan van Berkom <tristan.vanberkom@codethink.co.uk> | 2020-08-20 18:42:47 +0900 |
| commit | 41a7129139cd578876b3aefa08d9326cf789e38d (patch) | |
| tree | 92bd1ef044351bd29fd37fa33268d985173ae00b /tests/cachekey/project/sources/remote1.expected | |
| parent | 2e61b22df4ea5624d6368ca64103c78640744f58 (diff) | |
| download | buildstream-tristan/build-root-cache-key.tar.gz | |
buildstream/element.py: Consider "build-root" variable in the cache key unconditionallytristan/build-root-cache-key
Not all elements use the "build-root" variable, but it is the standard
variable to use for the build directory, and the build directory must
be considered in the cache key.
Handling this unilaterally in the core is safer than delegating this
to element implementations, as we have less chance of plugin authors
missing this detail and possibly introducing binary variance for
artifacts where only the build directory differs (something which
happens when the project name or element names change).
This commit also updates the hard coded cache keys in the cache key
test, so as to ensure every commit passes it's own tests.
This fixes #1386.
Diffstat (limited to 'tests/cachekey/project/sources/remote1.expected')
| -rw-r--r-- | tests/cachekey/project/sources/remote1.expected | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/cachekey/project/sources/remote1.expected b/tests/cachekey/project/sources/remote1.expected index 6f735034a..2c75c7c94 100644 --- a/tests/cachekey/project/sources/remote1.expected +++ b/tests/cachekey/project/sources/remote1.expected @@ -1 +1 @@ -94cac120225f3c023bdf3933b1248dbd1a0cdc401af1d0d76a3f603a1928c863
\ No newline at end of file +7e52b9c9005c0d6ea444c9297f6562b2eb1f8b7b0ed74fb4581ec6ccaae35101
\ No newline at end of file |