commit 1cfa9e3348bf1e53f246f540f8377b76eaa5aba1
author pcloudy <pcloudy@google.com> Wed Oct 30 05:07:08 2019 -0700
committer Copybara-Service <copybara-worker@google.com> Wed Oct 30 05:07:56 2019 -0700
tree a2f55273284c8071c440952090e3f4a504e9313a
parent 8aa2cf6495bee19ab4a1069432c854f6d0b2e797

TestStrategy: Truncate the tmp dir name to 32 character to avoid exceeding path length limit on Windows and macOS

Fixes https://github.com/bazelbuild/bazel/issues/10124

RELNOTES: None
PiperOrigin-RevId: 277483256

src/main/java/com/google/devtools/build/lib/exec/TestStrategy.java

diff --git a/src/main/java/com/google/devtools/build/lib/exec/TestStrategy.java b/src/main/java/com/google/devtools/build/lib/exec/TestStrategy.java
index f8c4dff..3430486 100644
--- a/src/main/java/com/google/devtools/build/lib/exec/TestStrategy.java
+++ b/src/main/java/com/google/devtools/build/lib/exec/TestStrategy.java
@@ -312,7 +312,8 @@
     digest.addPath(action.getExecutionSettings().getExecutable().getExecPath());
     digest.addInt(action.getShardNum());
     digest.addInt(action.getRunNumber());
-    return digest.hexDigestAndReset();
+    // Truncate the string to 32 character to avoid exceeding path length limit on Windows and macOS
+    return digest.hexDigestAndReset().substring(0, 32);
   }
 
   /** Parse a test result XML file into a {@link TestCase}. */