commit 8f1a9325e706768a0566cba995fd63f67101eee5
author Janak <janak@janak.org> Mon May 04 19:29:08 2015 +0000
committer Laurent Le Brun <laurentlb@google.com> Tue May 05 16:00:50 2015 +0000
tree 5cc396770839fa2a454dd6c90bef54ad5f0ebcab
parent b1231caae9638448a10011c1b520f8129762bb52

Adding additional logging that would have been useful when debugging issue #152.

--
Change-Id: I9968c3da2210b630502b47b7ed96acd71c7d7349
MOS_MIGRATED_REVID=92749744

src/main/java/com/google/devtools/build/lib/actions/Artifact.java

diff --git a/src/main/java/com/google/devtools/build/lib/actions/Artifact.java b/src/main/java/com/google/devtools/build/lib/actions/Artifact.java
index 4eb11ca..c39c228 100644
--- a/src/main/java/com/google/devtools/build/lib/actions/Artifact.java
+++ b/src/main/java/com/google/devtools/build/lib/actions/Artifact.java
@@ -139,10 +139,12 @@
   @VisibleForTesting
   public Artifact(Path path, Root root, PathFragment execPath, ArtifactOwner owner) {
     if (root == null || !path.startsWith(root.getPath())) {
-      throw new IllegalArgumentException(root + ": illegal root for " + path);
+      throw new IllegalArgumentException(root + ": illegal root for " + path
+          + " (execPath: " + execPath + ")");
     }
     if (execPath == null || execPath.isAbsolute() || !path.asFragment().endsWith(execPath)) {
-      throw new IllegalArgumentException(execPath + ": illegal execPath for " + path);
+      throw new IllegalArgumentException(execPath + ": illegal execPath for " + path
+          + " (root: " + root + ")");
     }
     this.path = path;
     this.root = root;