aboutsummaryrefslogtreecommitdiffhomepage
diff options
context:
space:
mode:
authorGravatar ulfjack <ulfjack@google.com>2018-07-19 03:35:09 -0700
committerGravatar Copybara-Service <copybara-piper@google.com>2018-07-19 03:36:37 -0700
commita12254f9d81569b71a99444b7c1690f7f28f1409 (patch)
tree2f48ac8757e627dbe0419574184a355066d42c8e
parent98670242d308595f9bc689781504176caa2e834b (diff)
Require the ActionExecutionMetadata to be passed to IncludeScanner
Also add an execution requirement that allows disabling reporting to the CLI; this will be used in a future change. PiperOrigin-RevId: 205216096
-rw-r--r--src/main/java/com/google/devtools/build/lib/actions/ExecutionRequirements.java3
-rw-r--r--src/main/java/com/google/devtools/build/lib/rules/cpp/IncludeScanner.java4
2 files changed, 7 insertions, 0 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/actions/ExecutionRequirements.java b/src/main/java/com/google/devtools/build/lib/actions/ExecutionRequirements.java
index f9e549ff77..2be9e27316 100644
--- a/src/main/java/com/google/devtools/build/lib/actions/ExecutionRequirements.java
+++ b/src/main/java/com/google/devtools/build/lib/actions/ExecutionRequirements.java
@@ -184,4 +184,7 @@ public class ExecutionRequirements {
* effect otherwise.
*/
public static final String REQUIRES_FAKEROOT = "requires-fakeroot";
+
+ /** Suppress CLI reporting for this spawn - it's part of another action. */
+ public static final String DO_NOT_REPORT = "internal-do-not-report";
}
diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/IncludeScanner.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/IncludeScanner.java
index 5ceb0a580d..18f4684c8e 100644
--- a/src/main/java/com/google/devtools/build/lib/rules/cpp/IncludeScanner.java
+++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/IncludeScanner.java
@@ -17,6 +17,7 @@ package com.google.devtools.build.lib.rules.cpp;
import com.google.common.collect.ImmutableList;
import com.google.common.collect.Sets;
import com.google.devtools.build.lib.actions.ActionExecutionContext;
+import com.google.devtools.build.lib.actions.ActionExecutionMetadata;
import com.google.devtools.build.lib.actions.Artifact;
import com.google.devtools.build.lib.actions.EnvironmentalExecException;
import com.google.devtools.build.lib.actions.ExecException;
@@ -70,6 +71,7 @@ public interface IncludeScanner {
IncludeScanningHeaderData includeScanningHeaderData,
List<String> cmdlineIncludes,
Set<Artifact> includes,
+ ActionExecutionMetadata actionExecutionMetadata,
ActionExecutionContext actionExecutionContext,
Artifact grepIncludes)
throws IOException, ExecException, InterruptedException;
@@ -104,6 +106,7 @@ public interface IncludeScanner {
IncludeScannable action,
IncludeScannerSupplier includeScannerSupplier,
IncludeScanningHeaderData includeScanningHeaderData,
+ ActionExecutionMetadata actionExecutionMetadata,
ActionExecutionContext actionExecutionContext,
String profilerTaskName)
throws ExecException, InterruptedException {
@@ -144,6 +147,7 @@ public interface IncludeScanner {
includeScanningHeaderData,
cmdlineIncludes,
includes,
+ actionExecutionMetadata,
actionExecutionContext,
action.getGrepIncludes());