Remove unnecessary injection of precomputed values. Precomputed values are injected in BlazeCommandDispatcher#execExclusively.

PiperOrigin-RevId: 169122571

1 files changed, 12 insertions, 16 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/runtime/CommandEnvironment.java b/src/main/java/com/google/devtools/build/lib/runtime/CommandEnvironment.java
index f656007bc5..9bb135cda5 100644
--- a/src/main/java/com/google/devtools/build/lib/runtime/CommandEnvironment.java
+++ b/src/main/java/com/google/devtools/build/lib/runtime/CommandEnvironment.java
@@ -501,22 +501,18 @@ public final class CommandEnvironment {
    */
   public void setupPackageCache(OptionsClassProvider options,
       String defaultsPackageContents) throws InterruptedException, AbruptExitException {
-    SkyframeExecutor skyframeExecutor = getSkyframeExecutor();
-
-    for (BlazeModule module : runtime.getBlazeModules()) {
-      skyframeExecutor.injectExtraPrecomputedValues(module.getPrecomputedValues());
-    }
-    skyframeExecutor.sync(
-        reporter,
-        options.getOptions(PackageCacheOptions.class),
-        options.getOptions(SkylarkSemanticsOptions.class),
-        getOutputBase(),
-        getWorkingDirectory(),
-        defaultsPackageContents,
-        getCommandId(),
-        clientEnv,
-        timestampGranularityMonitor,
-        options);
+    getSkyframeExecutor()
+        .sync(
+            reporter,
+            options.getOptions(PackageCacheOptions.class),
+            options.getOptions(SkylarkSemanticsOptions.class),
+            getOutputBase(),
+            getWorkingDirectory(),
+            defaultsPackageContents,
+            getCommandId(),
+            clientEnv,
+            timestampGranularityMonitor,
+            options);
   }
 
   public void recordLastExecutionTime() {