From ebf31dda35d08407cd6bfb822fc60e8002b97a18 Mon Sep 17 00:00:00 2001
From: Alexander Kriegisch <Alexander@Kriegisch.name>
Date: Thu, 28 Dec 2023 09:25:33 +0700
Subject: [PATCH] Move AspectJ Maven executions to default phases

Document the pros and cons in the POM with this comment:

Attention: aspectj-maven-plugin MUST be declared AFTER
maven-compiler-plugin, if they are both supposed to run in the same
default phases 'compile' and 'test-compile', but execution order is to
be guaranteed. Then, you have the convenience of being able to run e.g.
'mvn [clean] compile' instead of 'mvn [clean] process-classes'. For a
slightly less convenient, but less refactoring-errror-prone solution,
see the commented-out phases below.
---
 spring-data-jpa/pom.xml | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/spring-data-jpa/pom.xml b/spring-data-jpa/pom.xml
index 02133bbb33..189a62bd9a 100644
--- a/spring-data-jpa/pom.xml
+++ b/spring-data-jpa/pom.xml
@@ -415,6 +415,12 @@
 				</configuration>
 			</plugin>
 
+			<!--
+				Attention: aspectj-maven-plugin MUST be declared AFTER maven-compiler-plugin, if they are both supposed to run
+				in the same default phases 'compile' and 'test-compile', but execution order is to be guaranteed. Then, you have
+				the convenience of being able to run e.g. 'mvn [clean] compile' instead of 'mvn [clean] process-classes'. For a
+				slightly less convenient, but less refactoring-errror-prone solution, see the commented-out phases below.
+			-->
 			<plugin>
 				<groupId>org.codehaus.mojo</groupId>
 				<artifactId>aspectj-maven-plugin</artifactId>
@@ -432,14 +438,14 @@
 						<goals>
 							<goal>compile</goal>
 						</goals>
-						<phase>process-classes</phase>
+						<!--<phase>process-classes</phase>-->
 					</execution>
 					<execution>
 						<id>aspectj-test-compile</id>
 						<goals>
 							<goal>test-compile</goal>
 						</goals>
-						<phase>process-test-classes</phase>
+						<!--<phase>process-test-classes</phase>-->
 					</execution>
 				</executions>
 				<configuration>