Merge pull request #1198 from quickfix-j/copilot/check-os-specific-dependency

Remove OS-specific `netty-tcnative` by excluding it in POM

Author: chrjohn

pom.xml

@@ -98,9 +98,10 @@
 		<orchestra.file>OrchestraFIXLatest.xml</orchestra.file>
 		<org.quickfixj.orchestra.tools.version>1.0.3</org.quickfixj.orchestra.tools.version>
 		<jaxen.version>2.0.1</jaxen.version>
-        <jmh.version>1.37</jmh.version>
-        <jcstress.version>0.16</jcstress.version>
-        <hsqldb.version>1.8.0.10</hsqldb.version>
+    	<jmh.version>1.37</jmh.version>
+    	<jcstress.version>0.16</jcstress.version>
+    	<hsqldb.version>1.8.0.10</hsqldb.version>
+		<netty.version>4.2.13.Final</netty.version>
 		<class-pruner-plugin.version>3.0.1</class-pruner-plugin.version>
 	</properties>
 

quickfixj-core/pom.xml

@@ -120,8 +120,14 @@
 		<dependency>
 			<groupId>io.netty</groupId>
 			<artifactId>netty-example</artifactId>
-			<version>4.2.12.Final</version>
+			<version>${netty.version}</version>
 			<scope>test</scope>
+			<exclusions>
+				<exclusion>
+					<groupId>io.netty</groupId>
+					<artifactId>netty-tcnative</artifactId>
+				</exclusion>
+			</exclusions>
 		</dependency>
         <dependency>
             <groupId>org.burningwave</groupId>
@@ -231,14 +237,6 @@
 			</plugin>
 		</plugins>
 
-	  <extensions>
-		  <!-- required by Netty cross-platform build -->
-		  <extension>
-			  <groupId>kr.motd.maven</groupId>
-			  <artifactId>os-maven-plugin</artifactId>
-			  <version>1.7.1</version>
-		  </extension>
-	  </extensions>
 	</build>
 
 	<reporting>