From 2399984e69904f94223ef5d426986d390ccd06ad Mon Sep 17 00:00:00 2001
From: Jean-Francois Denise <jdenise@redhat.com>
Date: Tue, 17 Sep 2024 15:36:49 +0200
Subject: [PATCH 1/2] Shaded jar construction at provisioning time

---
 cli/assembly.xml                              | 21 +++++++++
 cli/pom.xml                                   | 45 +++++++++++++++++++
 core-feature-pack/common/pom.xml              | 12 -----
 .../packages/core.misc.common/package.xml     |  2 +
 .../main/resources/packages/tools/package.xml |  8 ++++
 .../packages/tools/pm/wildfly/tasks.xml       |  6 +--
 .../galleon-feature-pack/pom.xml              | 32 +++++++++++++
 elytron-tool-wrapper/assembly.xml             | 21 +++++++++
 elytron-tool-wrapper/pom.xml                  | 43 +++++++++++++-----
 pom.xml                                       |  8 +---
 10 files changed, 164 insertions(+), 34 deletions(-)
 create mode 100644 cli/assembly.xml
 create mode 100644 core-feature-pack/galleon-common/src/main/resources/packages/tools/package.xml
 create mode 100644 elytron-tool-wrapper/assembly.xml

diff --git a/cli/assembly.xml b/cli/assembly.xml
new file mode 100644
index 00000000000..d09a4ef3b08
--- /dev/null
+++ b/cli/assembly.xml
@@ -0,0 +1,21 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<assembly xmlns="http://maven.apache.org/plugins/maven-assembly-plugin/assembly/1.1.2" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+  xsi:schemaLocation="http://maven.apache.org/plugins/maven-assembly-plugin/assembly/1.1.2 http://maven.apache.org/xsd/assembly-1.1.2.xsd">
+    <id>shaded-model</id>
+    <formats>
+       <format>zip</format>
+    </formats>
+    <includeBaseDirectory>false</includeBaseDirectory>
+    <fileSets>
+        <fileSet>
+            <directory>target/resources</directory>
+            <outputDirectory/>
+        </fileSet>
+    </fileSets>
+    <files>
+        <file>
+            <source>pom.xml</source>
+            <outputDirectory/>
+        </file>
+    </files>
+</assembly>
diff --git a/cli/pom.xml b/cli/pom.xml
index 4076389fdad..78add71788b 100644
--- a/cli/pom.xml
+++ b/cli/pom.xml
@@ -58,6 +58,51 @@
                     </systemPropertyVariables>
                 </configuration>
             </plugin>
+            <plugin>
+                <groupId>org.wildfly.galleon-plugins</groupId>
+                <artifactId>wildfly-galleon-maven-plugin</artifactId>
+                <configuration>
+                    <mainClass>org.jboss.as.cli.CommandLineMain</mainClass>
+                    <manifestEntries>
+                        <Multi-Release>true</Multi-Release>
+                        <!-- Use sharable JPMS settings declared in the root pom -->
+                        <Add-Exports>${embedding.jar.jpms.exports}</Add-Exports>
+                        <Add-Opens>${embedding.jar.jpms.opens}</Add-Opens>
+                    </manifestEntries>
+                </configuration>
+                <executions>
+                    <execution>
+                        <phase>package</phase>
+                        <goals>
+                            <goal>generate-shaded-descriptor</goal>
+                        </goals>
+                    </execution>
+                </executions>
+            </plugin>
+            <plugin>
+                <groupId>org.apache.maven.plugins</groupId>
+                <artifactId>maven-assembly-plugin</artifactId>
+                <executions>
+                    <execution>
+                        <id>assemble</id>
+                        <phase>package</phase>
+                        <goals>
+                            <goal>single</goal>
+                        </goals>
+                        <configuration>
+                            <descriptors>
+                                <descriptor>assembly.xml</descriptor>
+                            </descriptors>
+                            <recompressZippedFiles>true</recompressZippedFiles>
+                            <finalName>${project.build.finalName}-shaded</finalName>
+                            <appendAssemblyId>true</appendAssemblyId>
+                            <outputDirectory>${project.build.directory}</outputDirectory>
+                            <workDirectory>${project.build.directory}/assembly/work</workDirectory>
+                            <tarLongFileMode>gnu</tarLongFileMode>
+                        </configuration>
+                    </execution>
+                </executions>
+            </plugin>
             <plugin>
                 <groupId>org.apache.maven.plugins</groupId>
                 <artifactId>maven-shade-plugin</artifactId>
diff --git a/core-feature-pack/common/pom.xml b/core-feature-pack/common/pom.xml
index f7834fb9d02..ecc5af38dba 100644
--- a/core-feature-pack/common/pom.xml
+++ b/core-feature-pack/common/pom.xml
@@ -635,18 +635,6 @@
             <artifactId>wildfly-cli</artifactId>
         </dependency>
 
-        <dependency>
-            <groupId>org.wildfly.core</groupId>
-            <artifactId>wildfly-cli</artifactId>
-            <classifier>client</classifier>
-            <exclusions>
-                <exclusion>
-                    <groupId>*</groupId>
-                    <artifactId>*</artifactId>
-                </exclusion>
-            </exclusions>
-        </dependency>
-
         <dependency>
             <groupId>org.wildfly.core</groupId>
             <artifactId>wildfly-deployment-scanner</artifactId>
diff --git a/core-feature-pack/galleon-common/src/main/resources/packages/core.misc.common/package.xml b/core-feature-pack/galleon-common/src/main/resources/packages/core.misc.common/package.xml
index 77714b07ed4..9487f3bc1fb 100644
--- a/core-feature-pack/galleon-common/src/main/resources/packages/core.misc.common/package.xml
+++ b/core-feature-pack/galleon-common/src/main/resources/packages/core.misc.common/package.xml
@@ -10,5 +10,7 @@
         <package name="LICENSE.txt"/>
         <package name="README.txt"/>
         <package name="copyright.txt"/>
+        <!-- required in all cases, used during provisioning time -->
+        <package name="org.wildfly.core.wildfly-cli.shaded"/>
     </dependencies>
 </package-spec>
\ No newline at end of file
diff --git a/core-feature-pack/galleon-common/src/main/resources/packages/tools/package.xml b/core-feature-pack/galleon-common/src/main/resources/packages/tools/package.xml
new file mode 100644
index 00000000000..4346e3dec48
--- /dev/null
+++ b/core-feature-pack/galleon-common/src/main/resources/packages/tools/package.xml
@@ -0,0 +1,8 @@
+<?xml version="1.0" ?>
+
+<package-spec xmlns="urn:jboss:galleon:package:2.0" name="tools">
+    <dependencies>
+        <package name="org.wildfly.core.wildfly-cli.shaded"/>
+        <package name="org.wildfly.core.wildfly-elytron-tool-wrapper.shaded"/>
+    </dependencies>
+</package-spec>
\ No newline at end of file
diff --git a/core-feature-pack/galleon-common/src/main/resources/packages/tools/pm/wildfly/tasks.xml b/core-feature-pack/galleon-common/src/main/resources/packages/tools/pm/wildfly/tasks.xml
index 1bad2d644eb..011483f6864 100644
--- a/core-feature-pack/galleon-common/src/main/resources/packages/tools/pm/wildfly/tasks.xml
+++ b/core-feature-pack/galleon-common/src/main/resources/packages/tools/pm/wildfly/tasks.xml
@@ -5,10 +5,10 @@
   ~ SPDX-License-Identifier: Apache-2.0
   -->
 
-<tasks xmlns="urn:wildfly:wildfly-feature-pack-tasks:2.0">
+<tasks xmlns="urn:wildfly:wildfly-feature-pack-tasks:3.2">
     <copy-artifact artifact="org.wildfly.core:wildfly-launcher" to-location="bin/launcher.jar"/>
-    <copy-artifact artifact="org.wildfly.core:wildfly-elytron-tool-wrapper" to-location="bin/wildfly-elytron-tool.jar"/>
-    <copy-artifact artifact="org.wildfly.core:wildfly-cli::client" to-location="bin/client/jboss-cli-client.jar"/>
+    <assemble-shaded-artifact shaded-model-package="org.wildfly.core.wildfly-elytron-tool-wrapper.shaded" to-location="bin/wildfly-elytron-tool.jar"/>
+    <assemble-shaded-artifact shaded-model-package="org.wildfly.core.wildfly-cli.shaded" to-location="bin/client/jboss-cli-client.jar"/>
     <file-permissions phase="FINALIZING">
         <permission value="755">
             <filter pattern="bin/client/jboss-cli-client.jar" include="true"/>
diff --git a/core-feature-pack/galleon-feature-pack/pom.xml b/core-feature-pack/galleon-feature-pack/pom.xml
index c5d4228a243..378d638f015 100644
--- a/core-feature-pack/galleon-feature-pack/pom.xml
+++ b/core-feature-pack/galleon-feature-pack/pom.xml
@@ -26,6 +26,8 @@
         <common.resources.directory>${basedir}/../common/src/main/resources</common.resources.directory>
         <ee-10-api.resources.directory>${basedir}/../ee-10-api/src/main/resources</ee-10-api.resources.directory>
         <galleon-common.resources.directory>${basedir}/../galleon-common/src/main/resources</galleon-common.resources.directory>
+        <galleon-cli-shaded.resources.directory>${basedir}/../../cli/target/resources</galleon-cli-shaded.resources.directory>
+        <galleon-elytron-tools-shaded.resources.directory>${basedir}/../../elytron-tool-wrapper/target/resources</galleon-elytron-tools-shaded.resources.directory>
     </properties>
 
     <dependencies>
@@ -175,6 +177,36 @@
                             </resources>
                         </configuration>
                     </execution>
+                    <execution>
+                        <id>copy-galleon-cli-shaded-resources</id>
+                        <phase>process-resources</phase>
+                        <goals>
+                            <goal>copy-resources</goal>
+                        </goals>
+                        <configuration>
+                            <outputDirectory>${basedir}/target/resources</outputDirectory>
+                            <resources>
+                                <resource>
+                                    <directory>${galleon-cli-shaded.resources.directory}</directory>
+                                </resource>
+                            </resources>
+                        </configuration>
+                    </execution>
+                    <execution>
+                        <id>copy-galleon-elytron-tools-shaded-resources</id>
+                        <phase>process-resources</phase>
+                        <goals>
+                            <goal>copy-resources</goal>
+                        </goals>
+                        <configuration>
+                            <outputDirectory>${basedir}/target/resources</outputDirectory>
+                            <resources>
+                                <resource>
+                                    <directory>${galleon-elytron-tools-shaded.resources.directory}</directory>
+                                </resource>
+                            </resources>
+                        </configuration>
+                    </execution>
                     <execution>
                         <id>copy-resources</id>
                         <phase>process-resources</phase>
diff --git a/elytron-tool-wrapper/assembly.xml b/elytron-tool-wrapper/assembly.xml
new file mode 100644
index 00000000000..d09a4ef3b08
--- /dev/null
+++ b/elytron-tool-wrapper/assembly.xml
@@ -0,0 +1,21 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<assembly xmlns="http://maven.apache.org/plugins/maven-assembly-plugin/assembly/1.1.2" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+  xsi:schemaLocation="http://maven.apache.org/plugins/maven-assembly-plugin/assembly/1.1.2 http://maven.apache.org/xsd/assembly-1.1.2.xsd">
+    <id>shaded-model</id>
+    <formats>
+       <format>zip</format>
+    </formats>
+    <includeBaseDirectory>false</includeBaseDirectory>
+    <fileSets>
+        <fileSet>
+            <directory>target/resources</directory>
+            <outputDirectory/>
+        </fileSet>
+    </fileSets>
+    <files>
+        <file>
+            <source>pom.xml</source>
+            <outputDirectory/>
+        </file>
+    </files>
+</assembly>
diff --git a/elytron-tool-wrapper/pom.xml b/elytron-tool-wrapper/pom.xml
index b21b0b2bc55..10ad87cb267 100644
--- a/elytron-tool-wrapper/pom.xml
+++ b/elytron-tool-wrapper/pom.xml
@@ -21,25 +21,44 @@
     <build>
         <plugins>
             <plugin>
-                <groupId>org.apache.maven.plugins</groupId>
-                <artifactId>maven-shade-plugin</artifactId>
+                <groupId>org.wildfly.galleon-plugins</groupId>
+                <artifactId>wildfly-galleon-maven-plugin</artifactId>
                 <configuration>
-                    <createDependencyReducedPom>false</createDependencyReducedPom>
-                    <transformers>
-                        <transformer implementation="org.apache.maven.plugins.shade.resource.ManifestResourceTransformer">
-                            <mainClass>org.wildfly.core.elytron.tool.wrapper.ElytronToolScriptWrapper</mainClass>
-                            <manifestEntries>
-                                <Multi-Release>true</Multi-Release>
-                            </manifestEntries>
-                        </transformer>
-                    </transformers>
+                    <mainClass>org.wildfly.core.elytron.tool.wrapper.ElytronToolScriptWrapper</mainClass>
+                    <manifestEntries>
+                        <Multi-Release>true</Multi-Release>
+                    </manifestEntries>
                 </configuration>
                 <executions>
                     <execution>
                         <phase>package</phase>
                         <goals>
-                            <goal>shade</goal>
+                            <goal>generate-shaded-descriptor</goal>
+                        </goals>
+                    </execution>
+                </executions>
+            </plugin>
+            <plugin>
+                <groupId>org.apache.maven.plugins</groupId>
+                <artifactId>maven-assembly-plugin</artifactId>
+                <executions>
+                    <execution>
+                        <id>assemble</id>
+                        <phase>package</phase>
+                        <goals>
+                            <goal>single</goal>
                         </goals>
+                        <configuration>
+                            <descriptors>
+                                <descriptor>assembly.xml</descriptor>
+                            </descriptors>
+                            <recompressZippedFiles>true</recompressZippedFiles>
+                            <finalName>${project.build.finalName}-shaded</finalName>
+                            <appendAssemblyId>true</appendAssemblyId>
+                            <outputDirectory>${project.build.directory}</outputDirectory>
+                            <workDirectory>${project.build.directory}/assembly/work</workDirectory>
+                            <tarLongFileMode>gnu</tarLongFileMode>
+                        </configuration>
                     </execution>
                 </executions>
             </plugin>
diff --git a/pom.xml b/pom.xml
index a41c6a8f418..ef3f90e2b55 100644
--- a/pom.xml
+++ b/pom.xml
@@ -127,7 +127,7 @@
         <version.jacoco.plugin>0.8.10</version.jacoco.plugin>
         <version.org.jboss.galleon>6.0.3.Final</version.org.jboss.galleon>
         <version.org.wildfly.checkstyle-config>1.0.8.Final</version.org.wildfly.checkstyle-config>
-        <version.org.wildfly.galleon-plugins>7.1.2.Final</version.org.wildfly.galleon-plugins>
+        <version.org.wildfly.galleon-plugins>7.1.3.Final-SNAPSHOT</version.org.wildfly.galleon-plugins>
         <version.org.eclipse.m2e.lifecycle-mapping>1.0.0</version.org.eclipse.m2e.lifecycle-mapping>
         <!-- wildfly-maven-plugin-->
         <version.wildfly.plugin>5.0.1.Final</version.wildfly.plugin>
@@ -1332,12 +1332,6 @@
                 <artifactId>wildfly-cli</artifactId>
                 <version>${project.version}</version>
             </dependency>
-            <dependency>
-                <groupId>org.wildfly.core</groupId>
-                <artifactId>wildfly-cli</artifactId>
-                <version>${project.version}</version>
-                <classifier>client</classifier>
-            </dependency>
             <dependency>
                 <groupId>org.wildfly.core</groupId>
                 <artifactId>wildfly-controller</artifactId>

From 6e6af87a57cfa17ba7c93c3ceda9bb2110be1a7d Mon Sep 17 00:00:00 2001
From: Jean-Francois Denise <jdenise@redhat.com>
Date: Wed, 18 Sep 2024 15:32:03 +0200
Subject: [PATCH 2/2] Add back cloent shaded jar dep

---
 core-feature-pack/common/pom.xml | 12 ++++++++++++
 pom.xml                          |  6 ++++++
 2 files changed, 18 insertions(+)

diff --git a/core-feature-pack/common/pom.xml b/core-feature-pack/common/pom.xml
index ecc5af38dba..f7834fb9d02 100644
--- a/core-feature-pack/common/pom.xml
+++ b/core-feature-pack/common/pom.xml
@@ -635,6 +635,18 @@
             <artifactId>wildfly-cli</artifactId>
         </dependency>
 
+        <dependency>
+            <groupId>org.wildfly.core</groupId>
+            <artifactId>wildfly-cli</artifactId>
+            <classifier>client</classifier>
+            <exclusions>
+                <exclusion>
+                    <groupId>*</groupId>
+                    <artifactId>*</artifactId>
+                </exclusion>
+            </exclusions>
+        </dependency>
+
         <dependency>
             <groupId>org.wildfly.core</groupId>
             <artifactId>wildfly-deployment-scanner</artifactId>
diff --git a/pom.xml b/pom.xml
index ef3f90e2b55..865c3fc2363 100644
--- a/pom.xml
+++ b/pom.xml
@@ -1332,6 +1332,12 @@
                 <artifactId>wildfly-cli</artifactId>
                 <version>${project.version}</version>
             </dependency>
+            <dependency>
+                <groupId>org.wildfly.core</groupId>
+                <artifactId>wildfly-cli</artifactId>
+                <version>${project.version}</version>
+                <classifier>client</classifier>
+            </dependency>
             <dependency>
                 <groupId>org.wildfly.core</groupId>
                 <artifactId>wildfly-controller</artifactId>