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
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 0532d72..97b6d36 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 @@
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
