Github user parthchandra commented on a diff in the pull request:
https://github.com/apache/drill/pull/991#discussion_r144960078
--- 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 --
OK. So I tried to put the commented out code in a 'openssl' profile and
discovered that maven does not allow the <extensions> tag inside a profile.
Since the problem is essentially caused by this extension being included, not
being able to put it in the 'openssl' profile makes this useless.
There is a sort of a convoluted hack around this described here:
https://stackoverflow.com/questions/17639778/maven-3-profile-with-extensions.
IMO this introduces a bit of unreadable, magical stuff in the pom which I
would rather avoid.
What do you think?
---
