aboutsummaryrefslogtreecommitdiffhomepage
path: root/src/main/java/com/google/devtools/build/lib/skyframe/SkyframeBuildView.java
diff options
context:
space:
mode:
authorGravatar tomlu <tomlu@google.com>2018-04-27 16:07:24 -0700
committerGravatar Copybara-Service <copybara-piper@google.com>2018-04-27 16:08:47 -0700
commit7f22066cbcebecb0a0b7133b907c0ef448825e6e (patch)
treed5420c6bcedd0c16e8baa9b4e5b9af0ca33aeda7 /src/main/java/com/google/devtools/build/lib/skyframe/SkyframeBuildView.java
parente793335d5f36f1728ff8445d59fbb49a2526cf84 (diff)
Output message when the build configuration changes.
RELNOTES: None PiperOrigin-RevId: 194607978
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/skyframe/SkyframeBuildView.java')
-rw-r--r--src/main/java/com/google/devtools/build/lib/skyframe/SkyframeBuildView.java10
1 files changed, 5 insertions, 5 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/skyframe/SkyframeBuildView.java b/src/main/java/com/google/devtools/build/lib/skyframe/SkyframeBuildView.java
index 115c5c1fa8..a86865822b 100644
--- a/src/main/java/com/google/devtools/build/lib/skyframe/SkyframeBuildView.java
+++ b/src/main/java/com/google/devtools/build/lib/skyframe/SkyframeBuildView.java
@@ -51,6 +51,7 @@ import com.google.devtools.build.lib.buildeventstream.BuildEventId;
import com.google.devtools.build.lib.cmdline.Label;
import com.google.devtools.build.lib.collect.nestedset.NestedSetBuilder;
import com.google.devtools.build.lib.events.Event;
+import com.google.devtools.build.lib.events.EventHandler;
import com.google.devtools.build.lib.events.ExtendedEventHandler;
import com.google.devtools.build.lib.packages.Attribute;
import com.google.devtools.build.lib.packages.NoSuchPackageException;
@@ -146,17 +147,16 @@ public final class SkyframeBuildView {
return factory;
}
- /**
- * Sets the configurations. Not thread-safe. DO NOT CALL except from tests!
- */
+ /** Sets the configurations. Not thread-safe. DO NOT CALL except from tests! */
@VisibleForTesting
- public void setConfigurations(BuildConfigurationCollection configurations) {
+ public void setConfigurations(
+ EventHandler eventHandler, BuildConfigurationCollection configurations) {
// Clear all cached ConfiguredTargets on configuration change of if --discard_analysis_cache
// was set on the previous build. In the former case, it's not required for correctness, but
// prevents unbounded memory usage.
if ((this.configurations != null && !configurations.equals(this.configurations))
|| skyframeAnalysisWasDiscarded) {
- logger.info("Discarding analysis cache: configurations have changed.");
+ eventHandler.handle(Event.info("Build options have changed, discarding analysis cache."));
skyframeExecutor.handleConfiguredTargetChange();
}
skyframeAnalysisWasDiscarded = false;