From 2f4a589f869a3667f60499a71eb24c6add533193 Mon Sep 17 00:00:00 2001 From: Oliver Reiche Date: Tue, 21 Feb 2023 16:33:05 +0100 Subject: ExecutionApi: Retrieve to correct CAS directly ... by keeping track of each blob being a file or executable and storing it to the correct local physical CAS directory. The new flag is merely a hint and only used by the local execution API. Leaving it out will still correctly transfer the blob but may cause unnecessary duplicates in file CAS. --- src/buildtool/execution_api/bazel_msg/blob_tree.cpp | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) (limited to 'src/buildtool/execution_api/bazel_msg/blob_tree.cpp') diff --git a/src/buildtool/execution_api/bazel_msg/blob_tree.cpp b/src/buildtool/execution_api/bazel_msg/blob_tree.cpp index 3e5e2fa6..88797970 100644 --- a/src/buildtool/execution_api/bazel_msg/blob_tree.cpp +++ b/src/buildtool/execution_api/bazel_msg/blob_tree.cpp @@ -69,8 +69,10 @@ auto BlobTree::FromDirectoryTree(DirectoryTreePtr const& tree, /*is_tree=*/true)); digest.set_size_bytes( gsl::narrow(git_tree->second.size())); - return std::make_shared( - BazelBlob{digest, git_tree->second}, nodes); + return std::make_shared(BazelBlob{digest, + git_tree->second, + /*is_exec=*/false}, + nodes); } } catch (...) { return std::nullopt; -- cgit v1.2.3