diff options
| author |  Janak Ramakrishnan <janakr@google.com> | 2015-08-31 22:19:07 +0000 |
|---|---|---|
| committer |  Kristina Chodorow <kchodorow@google.com> | 2015-09-01 00:21:00 +0000 |
| commit | 663f7c4ab9cfa4cb68f4f8ef83be7a66d1634da5 (patch) | |
| tree | 6d9c3575dd55b0b7893af0ef0147ad70374b6a3b /src/main/java/com/google/devtools/build | |
| parent | 7cdb722c6f59dd5fcc6f680a90ff4582e7a5e39f (diff) | |
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
Diffstat (limited to 'src/main/java/com/google/devtools/build')
| -rw-r--r-- | src/main/java/com/google/devtools/build/skyframe/ParallelEvaluator.java | 3 |
1 files changed, 1 insertions, 2 deletions
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.<SkyKey>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 |