Github user parthchandra commented on a diff in the pull request:
https://github.com/apache/drill/pull/991#discussion_r144666062
--- Diff: exec/java-exec/pom.xml ---
@@ -701,18 +707,21 @@
<!--
Include the os-maven-plugin to get os.detected.classifier
-->
- <extension>
- <groupId>kr.motd.maven</groupId>
- <artifactId>os-maven-plugin</artifactId>
- <version>1.5.0.Final</version>
- </extension>
- </extensions>
+ <!-- Uncomment the following to get a debug build that allows
openssl support -->
--- End diff --
How about we forget the whole os dependent stuff and put in all the
variants (there are only four) and make the scope for all of them `provided`?
Since netty is internally loading the correct jar based on the os, all we
need to do is make sure that the jar files are in the classpath.
---
