aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorEd Schouten <ed@nuxi.nl>2019-01-07 21:04:06 +0100
committerOla Rozenfeld <olaola@google.com>2019-01-07 15:04:06 -0500
commit78c169a474e24fd52c73c78fa7ee08295f39fe56 (patch)
treeaba5e43f36fca8bd1ecd2604ed3de385f4c702e4
parent4905c1a2a67a40f0e2659d84f1fcfd5101672679 (diff)
downloadbazelbuild-remote-apis-78c169a474e24fd52c73c78fa7ee08295f39fe56.tar.gz
Document that parent directories of outputs are created implicitly. (#42)
When implementing Buildbarn's worker, I noticed that there exists the assumption in build rules that directories under bazel-out/ are already created prior to execution, even if they are not part of the input root. As I suspect that getting rid of this assumption is both unrealistic and undesirable, let's document this instead. Mentioned in: #40
-rw-r--r--build/bazel/remote/execution/v2/remote_execution.proto7
1 files changed, 7 insertions, 0 deletions
diff --git a/build/bazel/remote/execution/v2/remote_execution.proto b/build/bazel/remote/execution/v2/remote_execution.proto
index d1dd383..9f177de 100644
--- a/build/bazel/remote/execution/v2/remote_execution.proto
+++ b/build/bazel/remote/execution/v2/remote_execution.proto
@@ -446,6 +446,9 @@ message Command {
//
// An output file cannot be duplicated, be a parent of another output file, or
// have the same path as any of the listed output directories.
+ //
+ // Directories leading up to the output files are created by the worker prior
+ // to execution, even if they are not explicitly part of the input root.
repeated string output_files = 3;
// A list of the output directories that the client expects to retrieve from
@@ -470,6 +473,10 @@ message Command {
//
// An output directory cannot be duplicated or have the same path as any of
// the listed output files.
+ //
+ // Directories leading up to the output directories (but not the output
+ // directories themselves) are created by the worker prior to execution, even
+ // if they are not explicitly part of the input root.
repeated string output_directories = 4;
// The platform requirements for the execution environment. The server MAY