From 4fbbb6977d73af9958e38bb07e096dee773c9e61 Mon Sep 17 00:00:00 2001 From: Maksim Denisov Date: Fri, 28 Feb 2025 16:41:21 +0100 Subject: ArtifactBlob: Support construction from temporary files --- src/buildtool/common/artifact_blob.cpp | 70 ++++++++++++++++++++++++++++++++++ 1 file changed, 70 insertions(+) (limited to 'src/buildtool/common/artifact_blob.cpp') diff --git a/src/buildtool/common/artifact_blob.cpp b/src/buildtool/common/artifact_blob.cpp index a9c70621..8c48d34e 100644 --- a/src/buildtool/common/artifact_blob.cpp +++ b/src/buildtool/common/artifact_blob.cpp @@ -91,12 +91,79 @@ auto ArtifactBlob::FromFile(HashFunction hash_function, } } +auto ArtifactBlob::FromTempFile(HashFunction hash_function, + ObjectType type, + TmpFile::Ptr file) noexcept + -> expected { + try { + if (file == nullptr) { + return unexpected{ + "ArtifactBlob::CreateFromTempFile: missing temp file."}; + } + + auto digest = IsTreeObject(type) + ? ArtifactDigestFactory::HashFileAs( + hash_function, file->GetPath()) + : ArtifactDigestFactory::HashFileAs( + hash_function, file->GetPath()); + if (not digest.has_value()) { + return unexpected{fmt::format( + "ArtifactBlob::CreateFromTempFile: Failed to hash {}", + file->GetPath().string())}; + } + return ArtifactBlob{ + *std::move(digest), std::move(file), IsExecutableObject(type)}; + } catch (const std::exception& e) { + return unexpected{fmt::format( + "ArtifactBlob::FromTempFile: Got an exception:\n{}", e.what())}; + } catch (...) { + return unexpected{ + "ArtifactBlob::FromTempFile: Got an unknown exception."}; + } +} + +auto ArtifactBlob::FromTempFile(HashFunction hash_type, + ObjectType type, + TmpDir::Ptr const& temp_space, + std::string const& content) noexcept + -> expected { + try { + if (temp_space == nullptr) { + return unexpected{ + "ArtifactBlob::FromTempFile: missing temp space."}; + } + + auto file = TmpDir::CreateFile(temp_space); + if (file == nullptr) { + return unexpected{ + "ArtifactBlob::FromTempFile: failed to create a new temp " + "file."}; + } + + if (not FileSystemManager::WriteFile(content, file->GetPath())) { + return unexpected{ + "ArtifactBlob::FromTempFile: failed to write content to a " + "temp file."}; + } + return FromTempFile(hash_type, type, std::move(file)); + } catch (const std::exception& e) { + return unexpected{fmt::format( + "ArtifactBlob::FromTempFile: Got an exception:\n{}", e.what())}; + } catch (...) { + return unexpected{ + "ArtifactBlob::FromTempFile: Got an unknown exception."}; + } +} + auto ArtifactBlob::ReadContent() const noexcept -> std::shared_ptr { using Result = std::shared_ptr; static constexpr InPlaceVisitor kVisitor{ [](InMemory const& value) -> Result { return value; }, [](InFile const& value) -> Result { return ::ReadFromFile(value); }, + [](InTempFile const& value) -> Result { + return ::ReadFromFile(value->GetPath()); + }, }; try { return std::visit(kVisitor, content_); @@ -119,6 +186,9 @@ auto ArtifactBlob::ReadIncrementally(std::size_t chunk_size) const& noexcept [chunk_size](InFile const& value) -> Result { return IncrementalReader::FromFile(chunk_size, value); }, + [chunk_size](InTempFile const& value) -> Result { + return IncrementalReader::FromFile(chunk_size, value->GetPath()); + }, }; try { return std::visit(visitor, content_); -- cgit v1.2.3