From 3bfc57aae4e9620d7620684b65a686fd5b045f20 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Christoph=20L=C3=A4ubrich?= <laeubi@laeubi-soft.de>
Date: Sat, 6 Jan 2024 13:45:32 +0100
Subject: [PATCH] Give the baseline artifact a better name and add extension

---
 .../core/osgitools/BaselineServiceImpl.java   | 23 +++++++++++++++----
 1 file changed, 18 insertions(+), 5 deletions(-)

diff --git a/tycho-core/src/main/java/org/eclipse/tycho/core/osgitools/BaselineServiceImpl.java b/tycho-core/src/main/java/org/eclipse/tycho/core/osgitools/BaselineServiceImpl.java
index cc049a7c68..2ebd4931f4 100644
--- a/tycho-core/src/main/java/org/eclipse/tycho/core/osgitools/BaselineServiceImpl.java
+++ b/tycho-core/src/main/java/org/eclipse/tycho/core/osgitools/BaselineServiceImpl.java
@@ -27,6 +27,7 @@
 import java.util.Map;
 import java.util.Map.Entry;
 
+import org.apache.commons.io.FilenameUtils;
 import org.codehaus.plexus.component.annotations.Component;
 import org.codehaus.plexus.component.annotations.Requirement;
 import org.codehaus.plexus.logging.Logger;
@@ -85,7 +86,8 @@ public Map<String, IP2Artifact> getProjectBaseline(Collection<MavenRepositoryLoc
         Map<String, IP2Artifact> result = new LinkedHashMap<>();
 
         for (Map.Entry<String, IP2Artifact> reactorArtifact : reactor.entrySet()) {
-            IArtifactDescriptor descriptor = reactorArtifact.getValue().getArtifactDescriptor();
+            IP2Artifact value = reactorArtifact.getValue();
+            IArtifactDescriptor descriptor = value.getArtifactDescriptor();
 
             Entry<IArtifactRepository, IArtifactDescriptor> baselineDescriptorEntry = getBaselineDescriptor(
                     baselineArtifacts, descriptor);
@@ -93,10 +95,10 @@ public Map<String, IP2Artifact> getProjectBaseline(Collection<MavenRepositoryLoc
                 continue;
             }
             IArtifactDescriptor baselineDescriptor = baselineDescriptorEntry.getValue();
-            IArtifactKey baslineKey = baselineDescriptor.getArtifactKey();
+            IArtifactKey baselineKey = baselineDescriptor.getArtifactKey();
             String format = baselineDescriptor.getProperty(IArtifactDescriptor.FORMAT);
-            File baselineArtifact = new File(target, baslineKey.getClassifier() + "/" + baslineKey.getId() + "/"
-                    + baslineKey.getVersion() + (format != null ? "." + format : ""));
+            File baselineArtifact = new File(target, baselineKey.getClassifier() + "/" + baselineKey.getId() + "-"
+                    + baselineKey.getVersion() + (format != null ? "." + format : "") + getExtension(value));
 
             baselineArtifact.getParentFile().mkdirs();
             try (OutputStream os = new BufferedOutputStream(new FileOutputStream(baselineArtifact))) {
@@ -114,7 +116,7 @@ public Map<String, IP2Artifact> getProjectBaseline(Collection<MavenRepositoryLoc
             }
 
             List<IInstallableUnit> units = new ArrayList<>();
-            for (IInstallableUnit unit : reactorArtifact.getValue().getInstallableUnits()) {
+            for (IInstallableUnit unit : value.getInstallableUnits()) {
                 IInstallableUnit baselineUnit = getBaselineUnit(baselineUnits, unit.getId(), unit.getVersion());
                 if (baselineUnit != null) {
                     units.add(baselineUnit);
@@ -126,6 +128,17 @@ public Map<String, IP2Artifact> getProjectBaseline(Collection<MavenRepositoryLoc
         return !result.isEmpty() ? result : null;
     }
 
+    private String getExtension(IP2Artifact value) {
+        File location = value.getLocation();
+        if (location != null) {
+            String extension = FilenameUtils.getExtension(location.getName());
+            if (!extension.isBlank()) {
+                return "." + extension;
+            }
+        }
+        return "";
+    }
+
     private Entry<IArtifactRepository, IArtifactDescriptor> getBaselineDescriptor(
             List<IArtifactRepository> baselineArtifacts, IArtifactDescriptor descriptor) {