diff options
author | Eric Fellheimer <felly@google.com> | 2015-05-05 21:52:13 +0000 |
---|---|---|
committer | Laszlo Csomor <laszlocsomor@google.com> | 2015-05-06 11:08:40 +0000 |
commit | ccd03539fd23375c8a20835c7ac048ce3e4717f0 (patch) | |
tree | 232a8f75f6cfc0b170ffbb014f0a4b6fc442be66 /src/main/java | |
parent | 0f172dba2828767b5061a0d09f26b8914bab3510 (diff) |
Defer injection of top-level context until we are just about to do the execution phase.
Commands which do not perform execution would otherwise over-invalidate nodes.
--
MOS_MIGRATED_REVID=92864280
Diffstat (limited to 'src/main/java')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/analysis/BuildView.java | 2 | ||||
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/buildtool/BuildTool.java | 1 |
2 files changed, 1 insertions, 2 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/analysis/BuildView.java b/src/main/java/com/google/devtools/build/lib/analysis/BuildView.java index 8ea61af5a1..8891b274a5 100644 --- a/src/main/java/com/google/devtools/build/lib/analysis/BuildView.java +++ b/src/main/java/com/google/devtools/build/lib/analysis/BuildView.java @@ -639,8 +639,6 @@ public class BuildView { filterTestsByTargets(configuredTargets, Sets.newHashSet(testsToRun))); } - skyframeExecutor.injectTopLevelContext(topLevelOptions); - Set<Artifact> artifactsToBuild = new HashSet<>(); Set<ConfiguredTarget> parallelTests = new HashSet<>(); Set<ConfiguredTarget> exclusiveTests = new HashSet<>(); diff --git a/src/main/java/com/google/devtools/build/lib/buildtool/BuildTool.java b/src/main/java/com/google/devtools/build/lib/buildtool/BuildTool.java index 9bc40fb2ca..d728c63db5 100644 --- a/src/main/java/com/google/devtools/build/lib/buildtool/BuildTool.java +++ b/src/main/java/com/google/devtools/build/lib/buildtool/BuildTool.java @@ -199,6 +199,7 @@ public class BuildTool { // Execution phase. if (needsExecutionPhase(request.getBuildOptions())) { + runtime.getSkyframeExecutor().injectTopLevelContext(request.getTopLevelArtifactContext()); executionTool.executeBuild(request.getId(), analysisResult, result, runtime.getSkyframeExecutor(), configurations, mergePackageRoots(loadingResult.getPackageRoots(), |