diff options
| author | Maksim Denisov <denisov.maksim@huawei.com> | 2025-02-25 13:42:22 +0100 |
|---|---|---|
| committer | Maksim Denisov <denisov.maksim@huawei.com> | 2025-02-27 09:03:30 +0100 |
| commit | 461312b57d3b49f92861d2c6c5e8a6b13ffa839b (patch) | |
| tree | c642cd0d3379e6886d1b3847d38661e249e75d58 /src/buildtool/execution_api/git | |
| parent | eccc7dcfb22fb9c6c42bbcd5566cd044acd1a2f3 (diff) | |
| download | justbuild-461312b57d3b49f92861d2c6c5e8a6b13ffa839b.tar.gz | |
ArtifactBlob: Use static function for construction
Diffstat (limited to 'src/buildtool/execution_api/git')
| -rw-r--r-- | src/buildtool/execution_api/git/git_api.cpp | 15 |
1 files changed, 6 insertions, 9 deletions
diff --git a/src/buildtool/execution_api/git/git_api.cpp b/src/buildtool/execution_api/git/git_api.cpp index 6021a913..231e717a 100644 --- a/src/buildtool/execution_api/git/git_api.cpp +++ b/src/buildtool/execution_api/git/git_api.cpp @@ -223,19 +223,16 @@ auto GitApi::RetrieveToCas( return false; } - ArtifactDigest digest = - IsTreeObject(info->type) - ? ArtifactDigestFactory::HashDataAs<ObjectType::Tree>( - hash_function, *content) - : ArtifactDigestFactory::HashDataAs<ObjectType::File>( - hash_function, *content); + auto blob = ArtifactBlob::FromMemory( + hash_function, info->type, *std::move(content)); + if (not blob.has_value()) { + return false; + } // Collect blob and upload to remote CAS if transfer size reached. if (not UpdateContainerAndUpload( &container, - ArtifactBlob{std::move(digest), - std::move(*content), - IsExecutableObject(info->type)}, + *std::move(blob), /*exception_is_fatal=*/true, [&api](std::unordered_set<ArtifactBlob>&& blobs) { return api.Upload(std::move(blobs), |