From e6914dc6382b92146b40a36d39a82c9f3226aad9 Mon Sep 17 00:00:00 2001
From: Henry Coles <henry@pitest.org>
Date: Wed, 8 May 2024 17:05:02 +0100
Subject: [PATCH 1/2] shade jackson

Version mismatches between jackson used on plugins can cause errors, so we are forced to shade it.
---
 pitest-aggregator/pom.xml | 35 ++++++++++++++++++++++++++++++++++-
 pom.xml                   |  2 +-
 2 files changed, 35 insertions(+), 2 deletions(-)

diff --git a/pitest-aggregator/pom.xml b/pitest-aggregator/pom.xml
index d08aa1bf6..62ec30813 100644
--- a/pitest-aggregator/pom.xml
+++ b/pitest-aggregator/pom.xml
@@ -13,6 +13,39 @@
 				<groupId>org.apache.maven.plugins</groupId>
 				<artifactId>maven-checkstyle-plugin</artifactId>
 			</plugin>
+			<plugin>
+				<artifactId>maven-shade-plugin</artifactId>
+				<executions>
+					<execution>
+						<phase>package</phase>
+						<goals>
+							<goal>shade</goal>
+						</goals>
+						<configuration>
+							<artifactSet>
+								<includes>
+									<include>com.fasterxml.jackson.dataformat:*</include>
+									<include>com.fasterxml.jackson.core:*</include>
+								</includes>
+							</artifactSet>
+							<relocations>
+								<relocation>
+									<pattern>com.fasterxml.jackson</pattern>
+									<shadedPattern>pitest.fasterxml.jackson</shadedPattern>
+								</relocation>
+							</relocations>
+							<!-- gradle scans the multi release jar includes with a potentially old version of ASM. Easiest to exclude.
+							This works for jackson, should be rechecked when including any off multi release dependencies-->
+							<exclude>META-INF/versions/**</exclude>
+						</configuration>
+					</execution>
+				</executions>
+			</plugin>
+			<!-- must run after shade -->
+			<plugin>
+				<groupId>org.codehaus.mojo</groupId>
+				<artifactId>flatten-maven-plugin</artifactId>
+			</plugin>
 		</plugins>
 	</build>
 	<dependencies>
@@ -29,7 +62,7 @@
 		<dependency>
 			<groupId>com.fasterxml.jackson.dataformat</groupId>
 			<artifactId>jackson-dataformat-xml</artifactId>
-			<version>2.16.2</version>
+			<version>2.17.0</version>
 		</dependency>
         <dependency>
             <groupId>junit</groupId>
diff --git a/pom.xml b/pom.xml
index c065e7cf0..7167f4789 100644
--- a/pom.xml
+++ b/pom.xml
@@ -322,7 +322,7 @@
 				<plugin>
 					<groupId>org.apache.maven.plugins</groupId>
 					<artifactId>maven-shade-plugin</artifactId>
-					<version>3.2.1</version>
+					<version>3.5.3</version>
 				</plugin>
 				<plugin>
 					<groupId>org.codehaus.mojo</groupId>

From 800a79500bd4fdb0a180079e51c0fcfec0bb2fe3 Mon Sep 17 00:00:00 2001
From: Henry Coles <henry@pitest.org>
Date: Wed, 8 May 2024 18:27:56 +0100
Subject: [PATCH 2/2] better error when aggregate goal run directly

---
 .../pitest/maven/report/AbstractPitAggregationReportMojo.java | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/pitest-maven/src/main/java/org/pitest/maven/report/AbstractPitAggregationReportMojo.java b/pitest-maven/src/main/java/org/pitest/maven/report/AbstractPitAggregationReportMojo.java
index b155c8f47..cfde317ef 100644
--- a/pitest-maven/src/main/java/org/pitest/maven/report/AbstractPitAggregationReportMojo.java
+++ b/pitest-maven/src/main/java/org/pitest/maven/report/AbstractPitAggregationReportMojo.java
@@ -75,6 +75,10 @@ protected void executeReport(final Locale locale)
     try {
       final Collection<MavenProject> allProjects = findDependencies();
 
+      if (allProjects.isEmpty()) {
+        throw new IllegalStateException("No projects found. Did you call the aggregate goal directly? It must be bound to a lifecycle phase");
+      }
+
       final ReportAggregator.Builder reportAggregationBuilder = ReportAggregator
           .builder();