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.

1 files changed, 4 insertions, 2 deletions

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<google::protobuf::int64>(git_tree->second.size()));
-            return std::make_shared<BlobTree>(
-                BazelBlob{digest, git_tree->second}, nodes);
+            return std::make_shared<BlobTree>(BazelBlob{digest,
+                                                        git_tree->second,
+                                                        /*is_exec=*/false},
+                                              nodes);
         }
     } catch (...) {
         return std::nullopt;