From 2cd16ea48276893c28064ddb91076afed327bd03 Mon Sep 17 00:00:00 2001
From: Clebert Suconic <clebertsuconic@apache.org>
Date: Wed, 6 Jun 2018 12:57:45 -0400
Subject: [PATCH] ARTEMIS-1913 Dependency to netty tcpnative should be optional

---
 artemis-core-client/pom.xml                      | 5 -----
 examples/features/standard/netty-openssl/pom.xml | 8 ++++++++
 pom.xml                                          | 7 -------
 tests/integration-tests/pom.xml                  | 6 ++++++
 tests/unit-tests/pom.xml                         | 6 ++++++
 5 files changed, 20 insertions(+), 12 deletions(-)

diff --git a/artemis-core-client/pom.xml b/artemis-core-client/pom.xml
index 092b8c399c5..47b72e23127 100644
--- a/artemis-core-client/pom.xml
+++ b/artemis-core-client/pom.xml
@@ -118,11 +118,6 @@
          <groupId>io.netty</groupId>
          <artifactId>netty-common</artifactId>
       </dependency>
-      <dependency>
-         <groupId>io.netty</groupId>
-         <artifactId>netty-tcnative-boringssl-static</artifactId>
-      </dependency>
-
    </dependencies>
 
    <profiles>
diff --git a/examples/features/standard/netty-openssl/pom.xml b/examples/features/standard/netty-openssl/pom.xml
index 5f61a20a12d..b146d280081 100644
--- a/examples/features/standard/netty-openssl/pom.xml
+++ b/examples/features/standard/netty-openssl/pom.xml
@@ -33,6 +33,7 @@ under the License.
 
    <properties>
       <activemq.basedir>${project.basedir}/../../../..</activemq.basedir>
+      <netty.tcnative.version>2.0.7.Final</netty.tcnative.version>
    </properties>
 
    <dependencies>
@@ -41,6 +42,12 @@ under the License.
          <artifactId>artemis-jms-client-all</artifactId>
          <version>${project.version}</version>
       </dependency>
+      <dependency>
+         <groupId>io.netty</groupId>
+         <artifactId>netty-tcnative-boringssl-static</artifactId>
+         <version>${netty.tcnative.version}</version>
+         <!-- License: Apache 2.0 -->
+      </dependency>
    </dependencies>
 
    <build>
@@ -56,6 +63,7 @@ under the License.
                   </goals>
                   <configuration>
                      <ignore>${noServer}</ignore>
+                     <libList>io.netty:netty-tcnative-boringssl-static:${netty.tcnative.version}</libList>
                   </configuration>
                </execution>
                <execution>
diff --git a/pom.xml b/pom.xml
index 8c506109644..7249c2dfb3e 100644
--- a/pom.xml
+++ b/pom.xml
@@ -92,7 +92,6 @@
       <maven.assembly.plugin.version>2.4</maven.assembly.plugin.version>
       <mockito.version>2.8.47</mockito.version>
       <netty.version>4.1.24.Final</netty.version>
-      <netty.tcnative.version>2.0.7.Final</netty.tcnative.version>
       <proton.version>0.27.1</proton.version>
       <resteasy.version>3.0.19.Final</resteasy.version>
       <slf4j.version>1.7.21</slf4j.version>
@@ -541,12 +540,6 @@
             <version>${netty.version}</version>
             <!-- License: Apache 2.0 -->
          </dependency>
-         <dependency>
-            <groupId>io.netty</groupId>
-            <artifactId>netty-tcnative-boringssl-static</artifactId>
-            <version>${netty.tcnative.version}</version>
-            <!-- License: Apache 2.0 -->
-         </dependency>
          <dependency>
             <groupId>org.apache.qpid</groupId>
             <artifactId>proton-j</artifactId>
diff --git a/tests/integration-tests/pom.xml b/tests/integration-tests/pom.xml
index d1c2b334498..ad278643ec7 100644
--- a/tests/integration-tests/pom.xml
+++ b/tests/integration-tests/pom.xml
@@ -397,6 +397,12 @@
          <artifactId>jgroups</artifactId>
       </dependency>
 
+      <!-- openSSL test -->
+      <dependency>
+         <groupId>io.netty</groupId>
+         <artifactId>netty-tcnative-boringssl-static</artifactId>
+         <version>2.0.7.Final</version>
+      </dependency>
    </dependencies>
 
    <build>
diff --git a/tests/unit-tests/pom.xml b/tests/unit-tests/pom.xml
index 5748163ddef..3f00b6ef107 100644
--- a/tests/unit-tests/pom.xml
+++ b/tests/unit-tests/pom.xml
@@ -160,6 +160,12 @@
          <version>2.7.0-SNAPSHOT</version>
          <scope>test</scope>
       </dependency>
+      <dependency>
+         <groupId>io.netty</groupId>
+         <artifactId>netty-tcnative-boringssl-static</artifactId>
+         <version>2.0.7.Final</version>
+         <scope>test</scope>
+      </dependency>
 
    </dependencies>