From e854c86bde5af363e03b87dcf46e629dce694c17 Mon Sep 17 00:00:00 2001 From: Googler Date: Fri, 18 May 2018 05:15:04 -0700 Subject: Changing SymlinkTreeStrategy not to use spawns. RELNOTES: An internal action for symlinking runfiles will use Command instead of a Spawns. This should have no functional chages; the only user visible consequence should be that the internal action is no longer be included in statistics when calculating processes count. PiperOrigin-RevId: 197131917 --- .../devtools/build/lib/analysis/actions/SymlinkTreeAction.java | 8 ++++---- .../build/lib/analysis/actions/SymlinkTreeActionContext.java | 4 +--- 2 files changed, 5 insertions(+), 7 deletions(-) (limited to 'src/main/java/com/google/devtools/build/lib/analysis') diff --git a/src/main/java/com/google/devtools/build/lib/analysis/actions/SymlinkTreeAction.java b/src/main/java/com/google/devtools/build/lib/analysis/actions/SymlinkTreeAction.java index 87e2130b86..10a30d8d9f 100644 --- a/src/main/java/com/google/devtools/build/lib/analysis/actions/SymlinkTreeAction.java +++ b/src/main/java/com/google/devtools/build/lib/analysis/actions/SymlinkTreeAction.java @@ -102,9 +102,9 @@ public final class SymlinkTreeAction extends AbstractAction { @Override public ActionResult execute(ActionExecutionContext actionExecutionContext) throws ActionExecutionException, InterruptedException { - return ActionResult.create( - actionExecutionContext - .getContext(SymlinkTreeActionContext.class) - .createSymlinks(this, actionExecutionContext, shellEnvironment, enableRunfiles)); + actionExecutionContext + .getContext(SymlinkTreeActionContext.class) + .createSymlinks(this, actionExecutionContext, shellEnvironment, enableRunfiles); + return ActionResult.EMPTY; } } diff --git a/src/main/java/com/google/devtools/build/lib/analysis/actions/SymlinkTreeActionContext.java b/src/main/java/com/google/devtools/build/lib/analysis/actions/SymlinkTreeActionContext.java index 449a0cc572..be6c9911cf 100644 --- a/src/main/java/com/google/devtools/build/lib/analysis/actions/SymlinkTreeActionContext.java +++ b/src/main/java/com/google/devtools/build/lib/analysis/actions/SymlinkTreeActionContext.java @@ -17,8 +17,6 @@ import com.google.common.collect.ImmutableMap; import com.google.devtools.build.lib.actions.ActionContext; import com.google.devtools.build.lib.actions.ActionExecutionContext; import com.google.devtools.build.lib.actions.ActionExecutionException; -import com.google.devtools.build.lib.actions.SpawnResult; -import java.util.List; /** * Action context for symlink tree actions (an action that creates a tree of symlinks). @@ -30,7 +28,7 @@ public interface SymlinkTreeActionContext extends ActionContext { * * @return a list of SpawnResults created during symlink creation, if any */ - List createSymlinks( + void createSymlinks( SymlinkTreeAction action, ActionExecutionContext actionExecutionContext, ImmutableMap shellEnvironment, -- cgit v1.2.3