commit bd864ef45a64b09d04978f3643a43f6b16e3877d
author jcater <jcater@google.com> Fri Jun 05 07:34:16 2020 -0700
committer Copybara-Service <copybara-worker@google.com> Fri Jun 05 07:36:12 2020 -0700
tree 2705d841837c98d772ddffea483a820f9c6ce7b2
parent 7b8b8144d1db3074c6ceefe6f802d12ab1c01031

Add a copy() method for Dependency.Builder.

Part of work on toolchain transitions, #10523.

PiperOrigin-RevId: 314925260

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

diff --git a/src/main/java/com/google/devtools/build/lib/analysis/Dependency.java b/src/main/java/com/google/devtools/build/lib/analysis/Dependency.java
index 87c650e..449a425 100644
--- a/src/main/java/com/google/devtools/build/lib/analysis/Dependency.java
+++ b/src/main/java/com/google/devtools/build/lib/analysis/Dependency.java
@@ -58,6 +58,10 @@
 
     /** Sets the keys of a configuration transition. */
     public Builder setTransitionKey(String key) {
+      if (key.isEmpty()) {
+        // Ignore empty keys.
+        return this;
+      }
       return setTransitionKeys(ImmutableList.of(key));
     }
 
@@ -77,6 +81,25 @@
       }
       return dependency;
     }
+
+    // Added to enable copy, below. Should not be accessible to other classes.
+    protected abstract Label getLabel();
+
+    @Nullable
+    protected abstract BuildConfiguration getConfiguration();
+
+    protected abstract AspectCollection getAspects();
+
+    protected abstract ImmutableList<String> getTransitionKeys();
+
+    /** Returns a copy of this Builder, with the values the same. */
+    public Builder copy() {
+      return Dependency.builder()
+          .setLabel(getLabel())
+          .setConfiguration(getConfiguration())
+          .setAspects(getAspects())
+          .setTransitionKeys(getTransitionKeys());
+    }
   }
 
   /** Returns a new {@link Builder} to create {@link Dependency} instances. */