From 116406fbbc665f6ea6d8fccf150389d808751642 Mon Sep 17 00:00:00 2001
From: Alexander Kriegisch <Alexander@Kriegisch.name>
Date: Fri, 16 Apr 2021 15:30:36 +0700
Subject: Fix: make sure that source assemblies are attached to build

Previously I renamed the source assemblies from the uniform name
'sources' to something more individual like 'aspectjtools-sources', not
realising that the magic name 'sources' in combination with the default
configuration value 'appendAssemblyId=true' results in an artifact
classifier equal to the assembly ID, i.e. 'sources', which is exactly
what we need here, but not quite obvious. Therefore, I documented it
with comments in both the assemblies and the POMs.

Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name>
---
 aspectjrt/aspectjrt-sources-assembly.xml | 3 ++-
 aspectjrt/pom.xml                        | 6 +++---
 2 files changed, 5 insertions(+), 4 deletions(-)

(limited to 'aspectjrt')

diff --git a/aspectjrt/aspectjrt-sources-assembly.xml b/aspectjrt/aspectjrt-sources-assembly.xml
index b4c13e2a9..fc030c3c2 100644
--- a/aspectjrt/aspectjrt-sources-assembly.xml
+++ b/aspectjrt/aspectjrt-sources-assembly.xml
@@ -2,7 +2,8 @@
 	xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
 	xsi:schemaLocation="http://maven.apache.org/ASSEMBLY/2.1.0 http://maven.apache.org/xsd/assembly-2.1.0.xsd">
 
-<id>aspectjrt-sources</id>
+	<!-- Do not change this ID. It becomes the artifact classifier with appendAssemblyId=true. -->
+	<id>sources</id>
 
 	<formats>
 		<format>jar</format>
diff --git a/aspectjrt/pom.xml b/aspectjrt/pom.xml
index 341b83fd6..930ba02ca 100644
--- a/aspectjrt/pom.xml
+++ b/aspectjrt/pom.xml
@@ -99,14 +99,14 @@
 					</execution>
 
 					<execution>
-						<id>aspectjrt-src-assembly</id>
+						<id>aspectjrt-sources-assembly</id>
 						<phase>package</phase>
 						<goals>
 							<goal>single</goal>
 						</goals>
 						<configuration>
-							<classifier>sources</classifier>
-							<finalName>aspectjrt-${project.version}</finalName>
+							<!-- The assembly ID 'sources' becomes the artifact classifier, exactly what we want -->
+							<appendAssemblyId>true</appendAssemblyId>
 							<archive>
 							</archive>
 							<descriptors>
-- 
cgit v1.2.3