diff options
author | Janak <janak@janak.org> | 2015-05-04 19:29:08 +0000 |
---|---|---|
committer | Laurent Le Brun <laurentlb@google.com> | 2015-05-05 16:00:50 +0000 |
commit | 8f1a9325e706768a0566cba995fd63f67101eee5 (patch) | |
tree | 5cc396770839fa2a454dd6c90bef54ad5f0ebcab | |
parent | b1231caae9638448a10011c1b520f8129762bb52 (diff) |
Adding additional logging that would have been useful when debugging issue #152.
--
Change-Id: I9968c3da2210b630502b47b7ed96acd71c7d7349
MOS_MIGRATED_REVID=92749744
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/actions/Artifact.java | 6 |
1 files changed, 4 insertions, 2 deletions
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 4eb11cad0c..c39c228bf3 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 @@ public class Artifact implements FileType.HasFilename, ActionInput { @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; |