Skip to content

Commit

Permalink
Upgrade dependencies to versions which use ASM 6.0.0+
Browse files Browse the repository at this point in the history
Motivation:

We need to upgrade our dependencies to versions which use ASM 6.0.0+ to support compiling on java9.

Modifications:

Update animal-sniffer-maven-plugin and maven-shade-plugin.

Result:

Fixes netty#6100
  • Loading branch information
johnou authored and normanmaurer committed Oct 7, 2017
1 parent 2fbe536 commit e805752
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 39 deletions.
24 changes: 0 additions & 24 deletions common/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -95,30 +95,6 @@
</configuration>
</execution>
</executions>
<!-- Needed to support JDK9 -->
<!-- See https://github.com/netty/netty/issues/6100 -->
<dependencies>
<dependency>
<groupId>org.ow2.asm</groupId>
<artifactId>asm</artifactId>
<version>${asm.version}</version>
</dependency>
<dependency>
<groupId>org.ow2.asm</groupId>
<artifactId>asm-commons</artifactId>
<version>${asm.version}</version>
</dependency>
<dependency>
<groupId>org.ow2.asm</groupId>
<artifactId>asm-analysis</artifactId>
<version>${asm.version}</version>
</dependency>
<dependency>
<groupId>org.ow2.asm</groupId>
<artifactId>asm-util</artifactId>
<version>${asm.version}</version>
</dependency>
</dependencies>
</plugin>
<plugin>
<groupId>org.apache.felix</groupId>
Expand Down
18 changes: 3 additions & 15 deletions pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -77,10 +77,6 @@
<argLine.java9>--add-modules java.xml.bind ${argLine.java9.extras}</argLine.java9>
<!-- Not use alpn agent as Java9 supports alpn out of the box -->
<argLine.alpnAgent />
<!-- This is needed as earlier ASM versions not support java9 -->
<!-- https://issues.apache.org/jira/browse/MSHADE-242 -->
<!-- https://github.com/netty/netty/issues/6100 -->
<asm.version>6.0_BETA</asm.version>
<!-- Skip as maven plugin not works with Java9 yet -->
<skipOsgiTestsuite>true</skipOsgiTestsuite>
<forbiddenapis.skip>true</forbiddenapis.skip>
Expand Down Expand Up @@ -258,7 +254,6 @@
<logging.config>${project.basedir}/../common/src/test/resources/logback-test.xml</logging.config>
<logging.logLevel>debug</logging.logLevel>
<log4j2.version>2.6.2</log4j2.version>
<asm.version>5.1</asm.version>
<enforcer.plugin.version>1.4.1</enforcer.plugin.version>
<testJavaHome>${env.JAVA_HOME}</testJavaHome>
<skipOsgiTestsuite>false</skipOsgiTestsuite>
Expand Down Expand Up @@ -634,15 +629,7 @@
be used even when compiling with java 1.7+ -->
<groupId>org.codehaus.mojo</groupId>
<artifactId>animal-sniffer-maven-plugin</artifactId>
<version>1.11</version>
<dependencies>
<!-- Upgrade ASM and support Java 8 bytecode -->
<dependency>
<groupId>org.ow2.asm</groupId>
<artifactId>asm-all</artifactId>
<version>${asm.version}</version>
</dependency>
</dependencies>
<version>1.16</version>
<configuration>
<signature>
<groupId>org.codehaus.mojo.signature</groupId>
Expand All @@ -657,6 +644,7 @@
<ignore>java.util.zip.Deflater</ignore>

<!-- Used for NIO -->
<ignore>java.net.ProtocolFamily</ignore>
<ignore>java.nio.channels.DatagramChannel</ignore>
<ignore>java.nio.channels.MembershipKey</ignore>
<ignore>java.nio.channels.ServerSocketChannel</ignore>
Expand Down Expand Up @@ -1155,7 +1143,7 @@
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-shade-plugin</artifactId>
<version>2.4.3</version>
<version>3.1.0</version>
</plugin>

<!-- Workaround for the 'M2E plugin execution not covered' problem.
Expand Down

0 comments on commit e805752

Please sign in to comment.