diff options
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/buildtool/BuildTool.java')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/buildtool/BuildTool.java | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/buildtool/BuildTool.java b/src/main/java/com/google/devtools/build/lib/buildtool/BuildTool.java index eab48fd588..777c9e8dbc 100644 --- a/src/main/java/com/google/devtools/build/lib/buildtool/BuildTool.java +++ b/src/main/java/com/google/devtools/build/lib/buildtool/BuildTool.java @@ -139,7 +139,6 @@ public final class BuildTool { env.setupPackageCache(request, DefaultsPackage.getDefaultsPackageContent(buildOptions)); ExecutionTool executionTool = null; - BuildConfigurationCollection configurations; boolean catastrophe = false; try { env.getEventBus().post(new BuildStartingEvent(env, request)); @@ -189,10 +188,9 @@ public final class BuildTool { env.throwPendingException(); // Configuration creation. - // TODO(gregce): BuildConfigurationCollection is important for static configs, less so for - // dynamic configs. Consider dropping it outright and passing on-the-fly target / host configs - // directly when needed (although this could be hard when Skyframe is unavailable). - configurations = + // TODO(gregce): Consider dropping this phase and passing on-the-fly target / host configs as + // needed. This requires cleaning up the invalidation in SkyframeBuildView.setConfigurations. + BuildConfigurationCollection configurations = env.getSkyframeExecutor() .createConfigurations( env.getReporter(), |