From 663f7c4ab9cfa4cb68f4f8ef83be7a66d1634da5 Mon Sep 17 00:00:00 2001 From: Janak Ramakrishnan Date: Mon, 31 Aug 2015 22:19:07 +0000 Subject: Don't pass temporary direct deps to environment when bubbling up. Normally, any known direct deps are guaranteed to be done. That's not true when bubbling. Since knowing no direct deps is a plausible state of affairs, just don't pass any. This may aid future code changes. -- MOS_MIGRATED_REVID=101980323 --- .../java/com/google/devtools/build/skyframe/ParallelEvaluator.java | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) (limited to 'src/main/java/com/google/devtools/build') diff --git a/src/main/java/com/google/devtools/build/skyframe/ParallelEvaluator.java b/src/main/java/com/google/devtools/build/skyframe/ParallelEvaluator.java index 6ddd647a14..88bd0ee22c 100644 --- a/src/main/java/com/google/devtools/build/skyframe/ParallelEvaluator.java +++ b/src/main/java/com/google/devtools/build/skyframe/ParallelEvaluator.java @@ -1248,8 +1248,7 @@ public final class ParallelEvaluator implements Evaluator { } } SkyFunctionEnvironment env = - new SkyFunctionEnvironment(parent, parentEntry.getTemporaryDirectDeps(), - bubbleErrorInfo, visitor); + new SkyFunctionEnvironment(parent, ImmutableSet.of(), bubbleErrorInfo, visitor); externalInterrupt = externalInterrupt || Thread.currentThread().isInterrupted(); try { // This build is only to check if the parent node can give us a better error. We don't -- cgit v1.2.3