commit 72e805be455bcb52c1f785f55978ec8c4d9296d7
author Janak Ramakrishnan <janakr@google.com> Fri Aug 21 18:03:57 2015 +0000
committer Han-Wen Nienhuys <hanwen@google.com> Mon Aug 24 14:02:14 2015 +0000
tree dbc3863170c0a7ca942c67fc8fbd6cd749dc414f
parent f3ec7c4ffd59936b668968d18865f80497e7d544

Preserve specified order of keys when analyzing configured targets.

--
MOS_MIGRATED_REVID=101235139

src/main/java/com/google/devtools/build/lib/skyframe/SkyframeExecutor.java

diff --git a/src/main/java/com/google/devtools/build/lib/skyframe/SkyframeExecutor.java b/src/main/java/com/google/devtools/build/lib/skyframe/SkyframeExecutor.java
index e188bf3..60d88a4 100644
--- a/src/main/java/com/google/devtools/build/lib/skyframe/SkyframeExecutor.java
+++ b/src/main/java/com/google/devtools/build/lib/skyframe/SkyframeExecutor.java
@@ -1162,16 +1162,13 @@
     return tsgm;
   }
 
-  /**
-   * Configures a given set of configured targets.
-   */
+  /** Configures a given set of configured targets. */
   public EvaluationResult<ActionLookupValue> configureTargets(
       List<ConfiguredTargetKey> values, List<AspectKey> aspectKeys, boolean keepGoing)
       throws InterruptedException {
     checkActive();
 
-    Set<SkyKey> keys = new HashSet<>();
-    keys.addAll(ConfiguredTargetValue.keys(values));
+    List<SkyKey> keys = new ArrayList<>(ConfiguredTargetValue.keys(values));
     for (AspectKey aspectKey : aspectKeys) {
       keys.add(AspectValue.key(aspectKey));
     }