diff options
author | 2015-10-16 11:58:26 +0000 | |
---|---|---|
committer | 2015-10-16 12:42:55 +0000 | |
commit | 4bcd39051873ac7dc50a717471cef3c28c648e33 (patch) | |
tree | d5649c96e463c738aac076c426d99ddf29a5ac9b /src/main/java | |
parent | c2fca389d4677cf321f22fb6a5ce2b6d7ed2e74a (diff) |
Fix reporting of error messages for EnvironmentalExecException
Most (all?) messages for this exception are complete description of the failure (e.g. "failed to do X", "could not do X"). With the previous wording the user got something like:
ERROR: [...] Writing file for rule '[...]' failedfailed to create file '[...]'
Now:
ERROR: [...] Writing file for rule '[...]' failed (failed to create file '[...]')
--
MOS_MIGRATED_REVID=105591532
Diffstat (limited to 'src/main/java')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/actions/EnvironmentalExecException.java | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/actions/EnvironmentalExecException.java b/src/main/java/com/google/devtools/build/lib/actions/EnvironmentalExecException.java index 6c1bc67f0a..f488728268 100644 --- a/src/main/java/com/google/devtools/build/lib/actions/EnvironmentalExecException.java +++ b/src/main/java/com/google/devtools/build/lib/actions/EnvironmentalExecException.java @@ -48,11 +48,11 @@ public class EnvironmentalExecException extends ExecException { public ActionExecutionException toActionExecutionException(String messagePrefix, boolean verboseFailures, Action action) { if (verboseFailures) { - return new ActionExecutionException(messagePrefix + " failed" + getMessage(), this, action, - isCatastrophic()); + return new ActionExecutionException( + messagePrefix + " failed (" + getMessage() + ")", this, action, isCatastrophic()); } else { - return new ActionExecutionException(messagePrefix + " failed" + getMessage(), action, - isCatastrophic()); + return new ActionExecutionException( + messagePrefix + " failed (" + getMessage() + ")", action, isCatastrophic()); } } } |