commit 317d5c008d9ef8b7f0e736cd72f578461019e648
author Romain Manni-Bucau <rmannibucau@gmail.com> Tue Nov 26 14:34:30 2019 +0100
committer Jean-Baptiste Onofré <jbonofre@apache.org> Tue Nov 26 14:46:53 2019 +0100
tree a0eed802eb7d8a46ec46dd59cb3d8edc256f2ac1
parent 314c01929131cfc170489c5d6ed13ac818510d10

ensure bundle order is respected in startup.properties

profile/src/main/java/org/apache/karaf/profile/assembly/Builder.java

diff --git a/profile/src/main/java/org/apache/karaf/profile/assembly/Builder.java b/profile/src/main/java/org/apache/karaf/profile/assembly/Builder.java
index a4d4205..7dd741b 100644
--- a/profile/src/main/java/org/apache/karaf/profile/assembly/Builder.java
+++ b/profile/src/main/java/org/apache/karaf/profile/assembly/Builder.java
@@ -109,6 +109,7 @@
 import org.slf4j.LoggerFactory;
 
 import static java.util.Collections.singletonList;
+import static java.util.Comparator.comparing;
 import static java.util.jar.JarFile.MANIFEST_NAME;
 import static org.apache.karaf.profile.assembly.Builder.Stage.Startup;
 
@@ -1962,7 +1963,10 @@
         Map<Integer, Set<String>> invertedStartupBundles = MapUtils.invert(bundles);
         for (Map.Entry<Integer, Set<String>> entry : new TreeMap<>(invertedStartupBundles).entrySet()) {
             String startLevel = Integer.toString(entry.getKey());
-            for (String location : new TreeSet<>(entry.getValue())) {
+            // ensure input order is respected whatever hashmap/set was in the middle of the processing
+            final List<String> value = new ArrayList<>(entry.getValue());
+            value.sort(comparing(bnd -> startupEffective.getBundles().indexOf(bnd)));
+            for (String location : value) {
                 if (useReferenceUrls) {
                     if (location.startsWith("mvn:")) {
                         location = "file:" + Parser.pathFromMaven(location);