ChenSammi commented on code in PR #10305:
URL: https://github.com/apache/ozone/pull/10305#discussion_r3264860564
##########
pom.xml:
##########
@@ -2633,5 +2633,126 @@
</plugins>
</build>
</profile>
+ <profile>
+ <id>native</id>
+ <activation>
+ <os>
+ <family>linux</family>
+ <arch>x86_64</arch>
+ </os>
+ <property>
+ <name>fetch-native-hadoop</name>
+ </property>
+ </activation>
Review Comment:
I did put it into dist pom.xml in first version. But there are at least two
more packages (unit test and integration test) requires native library to
run(not covered in this patch). So to avoid duplicate the profiles in multiple
pom.xml, gemini suggests to put it in root pom.xml, and with inherited false.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
