aboutsummaryrefslogtreecommitdiffhomepage
path: root/src/main/java/com/google/devtools/build/lib/actions/AbstractAction.java
diff options
context:
space:
mode:
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/actions/AbstractAction.java')
-rw-r--r--src/main/java/com/google/devtools/build/lib/actions/AbstractAction.java25
1 files changed, 14 insertions, 11 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/actions/AbstractAction.java b/src/main/java/com/google/devtools/build/lib/actions/AbstractAction.java
index 626b8731ad..7012f64034 100644
--- a/src/main/java/com/google/devtools/build/lib/actions/AbstractAction.java
+++ b/src/main/java/com/google/devtools/build/lib/actions/AbstractAction.java
@@ -479,20 +479,23 @@ public abstract class AbstractAction implements Action, SkylarkValue {
return true;
}
- /**
- * If the action might create directories as outputs this method must be called.
- */
- protected void checkOutputsForDirectories(EventHandler eventHandler) {
+ /** If the action might create directories as outputs this method must be called. */
+ protected void checkOutputsForDirectories(ActionExecutionContext actionExecutionContext) {
for (Artifact output : getOutputs()) {
- Path path = output.getPath();
+ Path path = actionExecutionContext.getInputPath(output);
String ownerString = Label.print(getOwner().getLabel());
if (path.isDirectory()) {
- eventHandler.handle(
- Event.warn(
- getOwner().getLocation(),
- "output '" + output.prettyPrint() + "' of " + ownerString
- + " is a directory; dependency checking of directories is unsound")
- .withTag(ownerString));
+ actionExecutionContext
+ .getEventHandler()
+ .handle(
+ Event.warn(
+ getOwner().getLocation(),
+ "output '"
+ + output.prettyPrint()
+ + "' of "
+ + ownerString
+ + " is a directory; dependency checking of directories is unsound")
+ .withTag(ownerString));
}
}
}