From 5b4ba3e9b5a3d2532e96812ed7f1fd87c263a467 Mon Sep 17 00:00:00 2001 From: Googler Date: Fri, 24 Nov 2023 03:00:31 -0800 Subject: [PATCH] StarlarkBaseExternalContext.java: propagate error message when deleting temporary directory failed Related: https://github.com/bazelbuild/bazel/issues/20013 PiperOrigin-RevId: 585046630 Change-Id: Ifdf098e7d54c1d5dca85a39afd7694dc828331a4 --- .../repository/starlark/StarlarkBaseExternalContext.java | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/main/java/com/google/devtools/build/lib/bazel/repository/starlark/StarlarkBaseExternalContext.java b/src/main/java/com/google/devtools/build/lib/bazel/repository/starlark/StarlarkBaseExternalContext.java index f34fb44603f63d..e68a62ded23707 100644 --- a/src/main/java/com/google/devtools/build/lib/bazel/repository/starlark/StarlarkBaseExternalContext.java +++ b/src/main/java/com/google/devtools/build/lib/bazel/repository/starlark/StarlarkBaseExternalContext.java @@ -727,7 +727,11 @@ public StructImpl downloadAndExtract( } catch (IOException e) { throw new RepositoryFunctionException( new IOException( - "Couldn't delete temporary directory (" + downloadDirectory.getPathString() + ")", e), + "Couldn't delete temporary directory (" + + downloadDirectory.getPathString() + + "): " + + e.getMessage(), + e), Transience.TRANSIENT); } return downloadResult;