commit 2ed5ed980e449b1d443c2886bb9b7015b748b23a
author shahan <shahan@google.com> Thu Sep 06 10:24:06 2018 -0700
committer Copybara-Service <copybara-piper@google.com> Thu Sep 06 10:25:34 2018 -0700
tree 7a6dbb7944c53216838eda1dfcffd892aa7cd495
parent 53a9e91fdc4f33288ffa60c3e66a836be884bc6b

Deletes @AutoCodec annotation from BuildConfiguration.Fragment subclasses.

It's less safe than DynamicCodec.

PiperOrigin-RevId: 211828418

src/main/java/com/google/devtools/build/lib/analysis/ShellConfiguration.java

diff --git a/src/main/java/com/google/devtools/build/lib/analysis/ShellConfiguration.java b/src/main/java/com/google/devtools/build/lib/analysis/ShellConfiguration.java
index 9e329ce..7da070a 100644
--- a/src/main/java/com/google/devtools/build/lib/analysis/ShellConfiguration.java
+++ b/src/main/java/com/google/devtools/build/lib/analysis/ShellConfiguration.java
@@ -20,7 +20,6 @@
 import com.google.devtools.build.lib.analysis.config.BuildOptions;
 import com.google.devtools.build.lib.analysis.config.ConfigurationFragmentFactory;
 import com.google.devtools.build.lib.analysis.config.FragmentOptions;
-import com.google.devtools.build.lib.skyframe.serialization.autocodec.AutoCodec;
 import com.google.devtools.build.lib.util.OS;
 import com.google.devtools.build.lib.util.OptionsUtils.PathFragmentConverter;
 import com.google.devtools.build.lib.vfs.PathFragment;
@@ -32,7 +31,6 @@
 import javax.annotation.Nullable;
 
 /** A configuration fragment that tells where the shell is. */
-@AutoCodec
 public class ShellConfiguration extends BuildConfiguration.Fragment {
   private static final ImmutableMap<OS, PathFragment> OS_SPECIFIC_SHELL =
       ImmutableMap.<OS, PathFragment>builder()
@@ -43,7 +41,7 @@
   private final PathFragment shellExecutable;
   private final boolean useShBinaryStubScript;
 
-  public ShellConfiguration(PathFragment shellExecutable, boolean useShBinaryStubScript) {
+  private ShellConfiguration(PathFragment shellExecutable, boolean useShBinaryStubScript) {
     this.shellExecutable = shellExecutable;
     this.useShBinaryStubScript = useShBinaryStubScript;
   }