From 385a17b10373b327c3a4b0033ce97faee1160a79 Mon Sep 17 00:00:00 2001
From: Norman Maurer <norman_maurer@apple.com>
Date: Thu, 5 Dec 2019 09:10:32 +0100
Subject: [PATCH] Include JCTools sources for shaded classes in the sources jar
 (#9838)

Motivation:

We should include the shaded sources for JCTools in our sources jar to make it easier to debug.

Modifications:

- Adjust plugin configuration to execute plugins in correct order
- Update source plugin
- Add configuration for shade plugin to generate source jar content

Result:

Fixes https://github.com/netty/netty/issues/6640.
---
 common/pom.xml |  2 ++
 pom.xml        | 19 ++++---------------
 2 files changed, 6 insertions(+), 15 deletions(-)

diff --git a/common/pom.xml b/common/pom.xml
index a187e69926..acfae05471 100644
--- a/common/pom.xml
+++ b/common/pom.xml
@@ -121,6 +121,8 @@
                 </relocation>
               </relocations>
               <minimizeJar>true</minimizeJar>
+              <createSourcesJar>true</createSourcesJar>
+              <shadeSourcesContent>true</shadeSourcesContent>
             </configuration>
           </execution>
         </executions>
diff --git a/pom.xml b/pom.xml
index 57adc5bdd2..383b3527a7 100644
--- a/pom.xml
+++ b/pom.xml
@@ -995,7 +995,7 @@
 
       <plugin>
         <artifactId>maven-source-plugin</artifactId>
-        <version>3.0.1</version>
+        <version>3.2.0</version>
         <!-- Eclipse-related OSGi manifests
              See https://github.com/netty/netty/issues/3886
              More information: http://rajakannappan.blogspot.ie/2010/03/automating-eclipse-source-bundle.html -->
@@ -1013,27 +1013,16 @@
         </configuration>
 
         <executions>
-          <!--
-            ~ This workaround prevents Maven from executing the 'generate-sources' phase twice.
-            ~ See http://jira.codehaus.org/browse/MSOURCES-13
-            ~ and http://blog.peterlynch.ca/2010/05/maven-how-to-prevent-generate-sources.html
-            -->
           <execution>
             <id>attach-sources</id>
-            <phase>invalid</phase>
-            <goals>
-              <goal>jar</goal>
-            </goals>
-          </execution>
-          <execution>
-            <id>attach-sources-no-fork</id>
-            <phase>package</phase>
+            <phase>prepare-package</phase>
             <goals>
               <goal>jar-no-fork</goal>
             </goals>
           </execution>
           <execution>
             <id>attach-test-sources</id>
+            <phase>prepare-package</phase>
             <goals>
               <goal>test-jar-no-fork</goal>
             </goals>
@@ -1295,7 +1284,7 @@
         <plugin>
           <groupId>org.apache.maven.plugins</groupId>
           <artifactId>maven-shade-plugin</artifactId>
-          <version>3.1.0</version>
+          <version>3.2.1</version>
         </plugin>
 
         <!-- Workaround for the 'M2E plugin execution not covered' problem.