diff options
author | tomlu <tomlu@google.com> | 2018-04-27 16:07:24 -0700 |
---|---|---|
committer | Copybara-Service <copybara-piper@google.com> | 2018-04-27 16:08:47 -0700 |
commit | 7f22066cbcebecb0a0b7133b907c0ef448825e6e (patch) | |
tree | d5420c6bcedd0c16e8baa9b4e5b9af0ca33aeda7 /src/main/java/com/google/devtools/build/lib/skyframe/SkyframeBuildView.java | |
parent | e793335d5f36f1728ff8445d59fbb49a2526cf84 (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.java | 10 |
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; |