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

Reply via email to