[maven-shade-plugin] branch fix-java16 created (now 56785aa)

[cs...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

cstamas pushed a change to branch fix-java16
in repository https://gitbox.apache.org/repos/asf/maven-shade-plugin.git.


      at 56785aa  Fix for Java16+

This branch includes the following new commits:

     new 56785aa  Fix for Java16+

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.

[maven-shade-plugin] 01/01: Fix for Java16+

[cs...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

cstamas pushed a commit to branch fix-java16
in repository https://gitbox.apache.org/repos/asf/maven-shade-plugin.git

commit 56785aa559870ff0906de8937198fb21178eea75
Author: Tamas Cservenak <ta...@cservenak.net>
AuthorDate: Fri May 21 10:07:21 2021 +0200

    Fix for Java16+
    
    The commit fa5e40dc59355163d89cd904fcf4c4724ba3d3d6
    made build fail on Java16 due illegal reflective access.
    
    The commit is wrong that is uses "aop" guice, that is
    the culprit of illegal reflective access (on Java11
    generates a warning on system out, but Java16
    prevents it).
    
    Solution: use the "no_aop" guice instead. Also,
    there was a SLF4J warning about non existence of
    backend in tests, fixed by adding simple backend
    with test scope.
---
 pom.xml | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/pom.xml b/pom.xml
index c428afb..a043736 100644
--- a/pom.xml
+++ b/pom.xml
@@ -107,6 +107,7 @@
         <groupId>org.sonatype.sisu</groupId>
         <artifactId>sisu-guice</artifactId>
         <version>3.2.6</version>
+        <classifier>no_aop</classifier>
       </dependency>
     </dependencies>
   </dependencyManagement>
@@ -228,6 +229,7 @@
     <dependency>
       <groupId>org.sonatype.sisu</groupId>
       <artifactId>sisu-guice</artifactId>
+      <classifier>no_aop</classifier>
       <scope>test</scope>
     </dependency>
     <dependency>
@@ -254,6 +256,12 @@
       <version>2.28.2</version>
       <scope>test</scope>
     </dependency>
+    <dependency>
+      <groupId>org.slf4j</groupId>
+      <artifactId>slf4j-simple</artifactId>
+      <version>1.7.30</version>
+      <scope>test</scope>
+    </dependency>
   </dependencies>
 
   <build>