diff options
author | Ed Schouten <ed@nuxi.nl> | 2019-01-07 21:04:06 +0100 |
---|---|---|
committer | Ola Rozenfeld <olaola@google.com> | 2019-01-07 15:04:06 -0500 |
commit | 78c169a474e24fd52c73c78fa7ee08295f39fe56 (patch) | |
tree | aba5e43f36fca8bd1ecd2604ed3de385f4c702e4 | |
parent | 4905c1a2a67a40f0e2659d84f1fcfd5101672679 (diff) | |
download | bazelbuild-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.proto | 7 |
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 |