diff options
author | Ulf Adams <ulfjack@google.com> | 2016-02-10 09:09:41 +0000 |
---|---|---|
committer | Dmitry Lomov <dslomov@google.com> | 2016-02-10 10:26:28 +0000 |
commit | 87935bb3abd7ca181c87fdc3fce83a4b02f4b394 (patch) | |
tree | 1da2b3f1139eb45d4ba0839cb5028d3c761b53e5 /src/main/java/com/google/devtools/build/lib | |
parent | 707b171601f4bf58028bd5186bc8618a0d9ae88b (diff) |
Fix callback handling in the Skyframe-based target pattern eval.
Add a unit test - there is test coverage for this in RunCommandTest, but this
runs much faster.
--
MOS_MIGRATED_REVID=114302062
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/skyframe/SkyframeExecutor.java | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/skyframe/SkyframeExecutor.java b/src/main/java/com/google/devtools/build/lib/skyframe/SkyframeExecutor.java index 291730adba..12c38a8cba 100644 --- a/src/main/java/com/google/devtools/build/lib/skyframe/SkyframeExecutor.java +++ b/src/main/java/com/google/devtools/build/lib/skyframe/SkyframeExecutor.java @@ -1789,6 +1789,9 @@ public abstract class SkyframeExecutor implements WalkableGraphFactory { eventBus.post(new TargetParsingCompleteEvent(patternParsingValue.getTargets(), patternParsingValue.getFilteredTargets(), patternParsingValue.getTestFilteredTargets(), time)); + if (callback != null) { + callback.notifyTargets(patternParsingValue.getTargets()); + } eventBus.post(new LoadingPhaseCompleteEvent( /*was expandedTargetsToLoad*/patternParsingValue.getTargets(), // TODO(ulfjack): Should be: Sets.difference(originalTargetsToLoad, expandedTargetsToLoad) |