Use the existing exception instance, not a new one.

Note that this can never happen in production - the loading phase ensures that
we only attempt to analyze targets that load successfully. That's also why
there are no tests for this.

This code path will become live in a later change, with corresponding test
coverage; the existing tests cover this in principle, we just need to make
them run with the reduced loading phase.

--
MOS_MIGRATED_REVID=112926211

1 files changed, 2 insertions, 3 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/skyframe/ConfiguredTargetFunction.java b/src/main/java/com/google/devtools/build/lib/skyframe/ConfiguredTargetFunction.java
index 0532d72990..97b6d36c75 100644
--- a/src/main/java/com/google/devtools/build/lib/skyframe/ConfiguredTargetFunction.java
+++ b/src/main/java/com/google/devtools/build/lib/skyframe/ConfiguredTargetFunction.java
@@ -155,9 +155,8 @@ final class ConfiguredTargetFunction implements SkyFunction {
     Target target;
     try {
       target = packageValue.getPackage().getTarget(lc.getLabel().getName());
-    } catch (NoSuchTargetException e1) {
-      throw new ConfiguredTargetFunctionException(new NoSuchTargetException(lc.getLabel(),
-          "No such target"));
+    } catch (NoSuchTargetException e) {
+      throw new ConfiguredTargetFunctionException(e);
     }
     transitivePackages.add(packageValue.getPackage());
     // TODO(bazel-team): This is problematic - we create the right key, but then end up with a value