uschindler commented on a change in pull request #18:
URL: https://github.com/apache/lucene/pull/18#discussion_r594202442
##########
File path: gradle/defaults-java.gradle
##########
@@ -60,3 +60,14 @@ allprojects {
}
}
}
+
+configure(project(":lucene:core")) {
+ plugins.withType(JavaPlugin) {
Review comment:
The problem is currently: it is incubating API, so it will break with
JDK17 again.
We have to wait that the incubating module get's into java.base /
java.whatever, so it can be used and API is stable. The problem with MR-JARs is
the following:
- it does not handle module system, so you would get ClassNotFound if you
dont start with correct command line flag
- You can't say: "use this only is Java version exactly 16"
Like with my own Pull Request for MMAP: We should test this, e.g. on
Jenkins, but not add to productive code.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
