Fix crash on multiple output symlinks with BwoB (#18226)

Fixes an issue introduced by ca30372. Fixes bazel-contrib/rules_go#3545 Closes #18207. PiperOrigin-RevId: 527160752 Change-Id: I8baa3b8f855e22226ba83155e56fd9889a1f2051 Co-authored-by: Fabian Meumertzheim <fabian@meumertzhe.im>
bazelbuild · Apr 26, 2023 · 3624c74 · 3624c74
1 parent d94dee2
commit 3624c74
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/main/java/com/google/devtools/build/lib/remote/RemoteExecutionService.java b/src/main/java/com/google/devtools/build/lib/remote/RemoteExecutionService.java
@@ -1254,7 +1254,7 @@ private boolean shouldDownloadOutputsFor(
               String.format(
                   "Symlinks in action outputs are not yet supported by --remote_download_minimal,"
                       + " falling back to downloading all action outputs due to output symlink %s",
-                  Iterables.getOnlyElement(metadata.symlinks()).path())));
+                  Iterables.get(metadata.symlinks(), 0).path())));
       return true;
     }
     return false;

diff --git a/src/test/shell/bazel/remote/build_without_the_bytes_test.sh b/src/test/shell/bazel/remote/build_without_the_bytes_test.sh
@@ -418,8 +418,8 @@ EOF
 genrule(
   name = "foo",
   srcs = ["input.txt"],
-  outs = ["output.txt", "output_symlink"],
-  cmd = "cp $< $(location :output.txt) && ln -s output.txt $(location output_symlink)",
+  outs = ["output.txt", "output_symlink", "output_symlink_2"],
+  cmd = "cp $< $(location :output.txt) && ln -s output.txt $(location output_symlink) && ln -s output.txt $(location output_symlink_2)",
 )
 EOF