summaryrefslogtreecommitdiff
path: root/src/buildtool/execution_engine/executor
diff options
context:
space:
mode:
authorSascha Roloff <sascha.roloff@huawei.com>2022-10-13 11:39:13 +0200
committerSascha Roloff <sascha.roloff@huawei.com>2022-10-14 12:46:42 +0200
commitf7e4ebf0e2913ff1b987e40baab5c0c809e6c0de (patch)
tree09ae121b46743a7eb682d9ba1361a59566c558bc /src/buildtool/execution_engine/executor
parent4d53961509015534fa10a59f4b86a20ae8a66017 (diff)
downloadjustbuild-f7e4ebf0e2913ff1b987e40baab5c0c809e6c0de.tar.gz
Remove default value of ArtifactDigest::Create template parameter
This enforces the explicit specification, which object type, either file or tree, should be used to create an artifact digest. This also avoids subtile errors at locations as in the previous commit, where files as well as trees are supposed to be handled, but digest creation mistakenly defaults to file object type.
Diffstat (limited to 'src/buildtool/execution_engine/executor')
-rw-r--r--src/buildtool/execution_engine/executor/executor.hpp2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/buildtool/execution_engine/executor/executor.hpp b/src/buildtool/execution_engine/executor/executor.hpp
index fc169200..ec280b01 100644
--- a/src/buildtool/execution_engine/executor/executor.hpp
+++ b/src/buildtool/execution_engine/executor/executor.hpp
@@ -382,7 +382,7 @@ class ExecutorImpl {
if (not content.has_value()) {
return std::nullopt;
}
- auto digest = ArtifactDigest::Create(*content);
+ auto digest = ArtifactDigest::Create<ObjectType::File>(*content);
if (not api->Upload(
BlobContainer{{BazelBlob{digest, std::move(*content)}}})) {
return std::nullopt;