From 9256e3ed52e0f7ba77e429a15118fef22a2f6960 Mon Sep 17 00:00:00 2001
From: Julien Viet <julien@julienviet.com>
Date: Thu, 22 Aug 2024 16:10:11 +0200
Subject: [PATCH] Declarative JPMS descriptors (#879)

Motivation:
tcnative should provide declarative modules for the set of its
artifacts. This is a preliminary contribution that will enable the support of
declarative JPMS descriptors for Netty 4.2 branch.

Modifications:
Generated artifacts are multi release jar to remain compatible with Java
8.

Result:
This has been tested with `netty-tcnative-openssl-static` on my arm64
mac with a jlink generated image.
---
 boringssl-static/pom.xml                      | 12 +++----
 libressl-static/pom.xml                       |  4 +--
 openssl-classes/src/main/java/module-info.yml |  2 ++
 openssl-dynamic/pom.xml                       |  4 +--
 openssl-static/pom.xml                        |  4 +--
 pom.xml                                       | 32 ++++++++++++++++++-
 6 files changed, 45 insertions(+), 13 deletions(-)
 create mode 100644 openssl-classes/src/main/java/module-info.yml

diff --git a/boringssl-static/pom.xml b/boringssl-static/pom.xml
index ff7060f1..0ba6d49f 100644
--- a/boringssl-static/pom.xml
+++ b/boringssl-static/pom.xml
@@ -63,7 +63,7 @@
     <ldflags>-L${boringsslHome}/ssl -L${boringsslHome}/crypto -lssl -lcrypto</ldflags>
     <skipJapicmp>true</skipJapicmp>
     <!-- We need to use the same module name for all our "native" impls as only one should be loaded -->
-    <javaModuleName>${javaModuleNameNative}</javaModuleName>
+    <javaModuleName>${javaDefaultModuleName}</javaModuleName>
   </properties>
 
   <dependencies>
@@ -277,7 +277,7 @@
                     </if>
 
                     <copy todir="${nativeJarWorkdir}">
-                      <zipfileset src="${defaultJarFile}" />
+                      <zipfileset src="${defaultJarFile}" excludes="META-INF/versions/**/module-info.class" />
                     </copy>
                     <copy todir="${nativeJarWorkdir}" includeEmptyDirs="false">
                       <zipfileset dir="${nativeLibOnlyDir}/META-INF/native" />
@@ -555,7 +555,7 @@
                     </if>
 
                     <copy todir="${nativeJarWorkdir}">
-                      <zipfileset src="${defaultJarFile}" />
+                      <zipfileset src="${defaultJarFile}" excludes="META-INF/versions/**/module-info.class" />
                     </copy>
                     <copy todir="${nativeJarWorkdir}" includeEmptyDirs="false">
                       <zipfileset dir="${nativeLibOnlyDir}/META-INF/native" />
@@ -890,7 +890,7 @@
                     </if>
 
                     <copy todir="${nativeJarWorkdir}">
-                      <zipfileset src="${defaultJarFile}" />
+                      <zipfileset src="${defaultJarFile}" excludes="META-INF/versions/**/module-info.class" />
                     </copy>
                     <copy todir="${nativeJarWorkdir}" includeEmptyDirs="false">
                       <zipfileset dir="${nativeLibOnlyDir}/META-INF/native" />
@@ -1309,7 +1309,7 @@
                     <taskdef resource="net/sf/antcontrib/antcontrib.properties" />
 
                     <copy todir="${nativeJarWorkdir}">
-                      <zipfileset src="${defaultJarFile}" />
+                      <zipfileset src="${defaultJarFile}" excludes="META-INF/versions/**/module-info.class" />
                     </copy>
                     <copy todir="${nativeJarWorkdir}" includeEmptyDirs="false">
                       <zipfileset dir="${nativeLibOnlyDir}/META-INF/native" />
@@ -1556,7 +1556,7 @@
                     <taskdef resource="net/sf/antcontrib/antcontrib.properties" />
 
                     <copy todir="${nativeJarWorkdir}">
-                      <zipfileset src="${defaultJarFile}" />
+                      <zipfileset src="${defaultJarFile}" excludes="META-INF/versions/**/module-info.class" />
                     </copy>
                     <copy todir="${nativeJarWorkdir}" includeEmptyDirs="false">
                       <zipfileset dir="${nativeLibOnlyDir}/META-INF/native" />
diff --git a/libressl-static/pom.xml b/libressl-static/pom.xml
index 7c6acab5..30f71423 100644
--- a/libressl-static/pom.xml
+++ b/libressl-static/pom.xml
@@ -40,7 +40,7 @@
     <maven.deploy.skip>true</maven.deploy.skip>
     <skipJapicmp>true</skipJapicmp>
     <!-- We need to use the same module name for all our "native" impls as only one should be loaded -->
-    <javaModuleName>${javaModuleNameNative}</javaModuleName>
+    <javaModuleName>${javaDefaultModuleName}</javaModuleName>
   </properties>
 
   <build>
@@ -112,7 +112,7 @@
             <configuration>
               <target>
                 <copy todir="${nativeJarWorkdir}">
-                  <zipfileset src="${defaultJarFile}" />
+                  <zipfileset src="${defaultJarFile}" excludes="META-INF/versions/**/module-info.class" />
                 </copy>
                 <copy todir="${nativeJarWorkdir}" includeEmptyDirs="false">
                   <zipfileset dir="${nativeLibOnlyDir}/META-INF/native" />
diff --git a/openssl-classes/src/main/java/module-info.yml b/openssl-classes/src/main/java/module-info.yml
new file mode 100644
index 00000000..56a03404
--- /dev/null
+++ b/openssl-classes/src/main/java/module-info.yml
@@ -0,0 +1,2 @@
+exports:
+  - package: io.netty.internal.tcnative
diff --git a/openssl-dynamic/pom.xml b/openssl-dynamic/pom.xml
index fb4b7152..c76b5e67 100644
--- a/openssl-dynamic/pom.xml
+++ b/openssl-dynamic/pom.xml
@@ -36,7 +36,7 @@
     <nativeSourceDirectory />
     <skipJapicmp>true</skipJapicmp>
     <!-- We need to use the same module name for all our "native" impls as only one should be loaded -->
-    <javaModuleName>${javaModuleNameNative}</javaModuleName>
+    <javaModuleName>${javaDefaultModuleName}</javaModuleName>
   </properties>
 
   <build>
@@ -77,7 +77,7 @@
             <configuration>
               <target>
                 <copy todir="${nativeJarWorkdir}">
-                  <zipfileset src="${defaultJarFile}" />
+                  <zipfileset src="${defaultJarFile}" excludes="META-INF/versions/**/module-info.class" />
                 </copy>
                 <copy todir="${nativeJarWorkdir}" includeEmptyDirs="false">
                   <zipfileset dir="${nativeLibOnlyDir}/META-INF/native" />
diff --git a/openssl-static/pom.xml b/openssl-static/pom.xml
index 485ccb3b..fb6b0fdb 100644
--- a/openssl-static/pom.xml
+++ b/openssl-static/pom.xml
@@ -39,7 +39,7 @@
     <maven.deploy.skip>true</maven.deploy.skip>
     <skipJapicmp>true</skipJapicmp>
     <!-- We need to use the same module name for all our "native" impls as only one should be loaded -->
-    <javaModuleName>${javaModuleNameNative}</javaModuleName>
+    <javaModuleName>${javaDefaultModuleName}</javaModuleName>
   </properties>
 
   <build>
@@ -109,7 +109,7 @@
             <configuration>
               <target>
                 <copy todir="${nativeJarWorkdir}">
-                  <zipfileset src="${defaultJarFile}" />
+                  <zipfileset src="${defaultJarFile}" excludes="META-INF/versions/**/module-info.class" />
                 </copy>
                 <copy todir="${nativeJarWorkdir}" includeEmptyDirs="false">
                   <zipfileset dir="${nativeLibOnlyDir}/META-INF/native" />
diff --git a/pom.xml b/pom.xml
index 3beb8a36..aa1153ed 100644
--- a/pom.xml
+++ b/pom.xml
@@ -233,6 +233,36 @@
         </configuration>
       </plugin>
 
+      <plugin>
+        <groupId>io.github.dmlloyd.module-info</groupId>
+        <artifactId>module-info</artifactId>
+        <version>2.1</version>
+        <executions>
+          <execution>
+            <id>default-jar-module-info</id>
+            <phase>process-classes</phase>
+            <configuration>
+              <moduleName>${javaModuleName}</moduleName>
+              <outputDirectory>${project.build.outputDirectory}/META-INF/versions/11/</outputDirectory>
+            </configuration>
+            <goals>
+              <goal>generate</goal>
+            </goals>
+          </execution>
+          <execution>
+            <id>native-jar-module-info</id>
+            <phase>process-classes</phase>
+            <configuration>
+              <moduleName>${javaModuleNameNative}</moduleName>
+              <outputDirectory>${nativeJarWorkdir}//META-INF/versions/11/</outputDirectory>
+            </configuration>
+            <goals>
+              <goal>generate</goal>
+            </goals>
+          </execution>
+        </executions>
+      </plugin>
+
       <plugin>
         <artifactId>maven-surefire-plugin</artifactId>
         <version>2.22.2</version>
@@ -439,7 +469,7 @@
                   <addDefaultSpecificationEntries>true</addDefaultSpecificationEntries>
                 </manifest>
                 <manifestEntries>
-                  <Automatic-Module-Name>${javaModuleName}</Automatic-Module-Name>
+                  <Multi-Release>true</Multi-Release>
                 </manifestEntries>
                 <index>true</index>
                 <manifestFile>${project.build.outputDirectory}/META-INF/MANIFEST.MF</manifestFile>