commit 1159cc2a43c4f6416c11e13b115fc26c63e769c5
author Laurent Le Brun <laurentlb@google.com> Wed Jan 04 12:27:39 2017 +0000
committer John Cater <jcater@google.com> Wed Jan 04 20:38:06 2017 +0000
tree 828f6be988194970ff6de5c4831b2573d4fdf748
parent 9340550de931d952055f3d7b95fd09b96b5bed6c

Minor improvements to error messages.

In case of conflict, show the location of the original rule.

--
PiperOrigin-RevId: 143541281
MOS_MIGRATED_REVID=143541281

src/main/java/com/google/devtools/build/lib/packages/Package.java

diff --git a/src/main/java/com/google/devtools/build/lib/packages/Package.java b/src/main/java/com/google/devtools/build/lib/packages/Package.java
index 8296c36..a7482bb 100644
--- a/src/main/java/com/google/devtools/build/lib/packages/Package.java
+++ b/src/main/java/com/google/devtools/build/lib/packages/Package.java
@@ -1509,12 +1509,14 @@
     private static String conflictsWith(Target target) {
       String message = "conflicts with existing ";
       if (target instanceof OutputFile) {
-        return message + "generated file from rule '"
-          + ((OutputFile) target).getGeneratingRule().getName()
-          + "'";
+        message +=
+            "generated file from rule '"
+                + ((OutputFile) target).getGeneratingRule().getName()
+                + "'";
       } else {
-        return message + target.getTargetKind();
+        message += target.getTargetKind();
       }
+      return message + ", defined at " + target.getLocation();
     }
   }
 }