mocobeta commented on PR #941: URL: https://github.com/apache/lucene/pull/941#issuecomment-1146089576
> So I am fine with current patch, but the MAXIMUM version must be 17 also on 9.x (11 till 17). We have another pull request #943 for 9x and the maximum major is also set to 17. > I think the checks should be in the alternate JVM support gradle file. We're not talking here about Gradle runs/won't run, we are talking here about the Java version to execute javac/javadoc/tests. And this one is not necessarily the one of Gradle so the check is at wrong place. Ok, It'd be an abusive use of the `checkVersion()` to prevent developers (or release managers) to use too old Java - honestly I know it. I will take a look at the way to do this in proper gradle scripts. > P.P.S: Please wait with this until after Berlinbuzzwords. I am so busy at moment, so I can't take care of Jenkins. But maybe set lower limit to 17.0.2 and 11.0.X (X unknown). Updating can be done from time to time if there are bugs hitting us. We should not require latest, this may also prevent new contributors from starting. Thanks, I don't think we are not in hurry at all, and I'm not intend to ask to upgrade Jenkins servers now - I just needed to know how to proceed this without invading CI jobs. -- 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: issues-unsubscr...@lucene.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org
