From 7010da7ce5cab6fa675716e85d84fb5c0ff1a558 Mon Sep 17 00:00:00 2001
From: Michael Keppler <Michael.Keppler@gmx.de>
Date: Mon, 23 Dec 2024 09:39:57 +0100
Subject: [PATCH] Simplify patch creation

Revert most of aab82332b0fe1953209f8a1955bc79bfba25f4bb. Instead of
using a profile, just add <inherited>false</inherited> to the plugin
execution we don't want in the child modules.
---
 pom.xml | 63 +++++++++++++++++++++------------------------------------
 1 file changed, 23 insertions(+), 40 deletions(-)

diff --git a/pom.xml b/pom.xml
index 9db50e300..5256f4704 100644
--- a/pom.xml
+++ b/pom.xml
@@ -138,6 +138,29 @@
                     </environments>
                 </configuration>
             </plugin>
+            <plugin>
+                <groupId>org.codehaus.mojo</groupId>
+                <artifactId>exec-maven-plugin</artifactId>
+                <executions>
+                    <execution>
+                        <id>create-git-patch-file</id>
+                        <phase>verify</phase>
+                        <goals>
+                            <goal>exec</goal>
+                        </goals>
+                    </execution>
+                </executions>
+                <configuration>
+                    <executable>git</executable>
+                    <arguments>
+                        <argument>diff</argument>
+                        <argument>HEAD~1</argument>
+                        <argument>HEAD</argument>
+                    </arguments>
+                    <outputFile>${maven.multiModuleProjectDirectory}/show.patch</outputFile>
+                </configuration>
+                <inherited>false</inherited>
+            </plugin>
             <plugin>
                 <artifactId>maven-checkstyle-plugin</artifactId>
                 <executions>
@@ -517,44 +540,4 @@
             </plugins>
         </pluginManagement>
     </build>
-    <profiles>
-        <profile>
-            <id>git-patch</id>
-            <!-- This POM is both parent and reactor, therefore all plugins in the build section will be executed for all modules.
-            By checking for the existence of the README, we limit the execution of this profile to the parent only, thereby creating the patch only once.
-            TODO: In the long run, we should probably split parent and reactor into 2 POM files instead. -->
-            <activation>
-                <file>
-                    <exists>README.md</exists>
-                </file>
-            </activation>
-            <build>
-                <plugins>
-                    <plugin>
-                        <groupId>org.codehaus.mojo</groupId>
-                        <artifactId>exec-maven-plugin</artifactId>
-                        <executions>
-                            <execution>
-                                <id>create-git-patch-file</id>
-                                <!-- must be earlier than the phase we bind the checkstyle goal to -->
-                                <phase>package</phase>
-                                <goals>
-                                    <goal>exec</goal>
-                                </goals>
-                            </execution>
-                        </executions>
-                        <configuration>
-                            <executable>git</executable>
-                            <arguments>
-                                <argument>diff</argument>
-                                <argument>HEAD~1</argument>
-                                <argument>HEAD</argument>
-                            </arguments>
-                            <outputFile>${maven.multiModuleProjectDirectory}/show.patch</outputFile>
-                        </configuration>
-                    </plugin>
-                </plugins>
-            </build>
-        </profile>
-    </profiles>
 </project>