rmuir commented on code in PR #14651: URL: https://github.com/apache/lucene/pull/14651#discussion_r2086560622
########## gradle/validation/ecj-lint.gradle: ########## @@ -94,10 +96,10 @@ allprojects { args += [ "-d", "none" ] + assert project.java.sourceCompatibility == project.java.targetCompatibility + // Compilation environment. - // we use -source/-target as it is significantly faster than --release - args += [ "-source", project.java.sourceCompatibility ] - args += [ "-target", project.java.targetCompatibility ] + args += ["-$project.java.sourceCompatibility"] // "compliance" level Review Comment: i dont think we need to do any CLI flags change: That's exactly what I'm requesting to revert here. And thats why the PR has 40+ comments. If we were doing this with `make` we'd just `envsubst` and the hardcoded value would be replaced by variable: problem solved. I don't understand why it should be any more difficult than that with gradle. -- 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