jochenw commented on a change in pull request #268:
URL: https://github.com/apache/commons-io/pull/268#discussion_r707160987
##########
File path: pom.xml
##########
@@ -600,5 +630,13 @@ file comparators, endian transformation classes, and much
more.
</plugins>
</build>
</profile>
+ <profile>
+ <id>release</id>
+ <properties>
+ <!-- Enable Moditect for releases. This requires JDK 9+ for the build
-->
Review comment:
@garydgregory I think, that **release building** with Java 9+ is a more
than reasonable requirement, if we want to fit into the Java 9 ecosphere.
(There is no question, that we want that, isn't there?)
@martin-g I'd like to see two improvements:
1. Move the plugin definition into a profile, that enables the
moditect-maven-plugin only when building with Java 9+. Alternatively, set the
property moditect.skip for building with Java 8.
2. Make sure, that the apache-release profile enforces building with Java 9+.
Regarding commons-parent: My personal stance is, that we have covered
commons-io, and commons-fileupload now (Perhaps, cover a few others, in case of
demand.) Let's wait, and see, until we did a release with that, then approach
parent.
--
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]
