Build: remove hard-coded Java versions from ecj.javadocs.prefs (#14651)

And retain correct Eclipse IDE configuration generation by templating org.eclipse.jdt.core.prefs with the JDK version.

---------

Co-authored-by: Uwe Schindler <[email protected]>

(cherry picked from commit 78da2ee)

Author: dsmiley

gradle/ide/eclipse.gradle

@@ -96,7 +96,8 @@ configure(rootProject) {
         from rootProject.file("${resources}/dot.settings")
         into rootProject.file(".settings")
         filter(ReplaceTokens, tokens: [
-                'ecj-lint-config': ecjLintFile.getText('UTF-8').replaceAll(/=error\b/, '=' + errorMode)
+                'ecj-lint-config': ecjLintFile.getText('UTF-8').replaceAll(/=error\b/, '=' + errorMode),
+                'eclipseJavaVersion': eclipseJavaVersion as String
         ])
         filteringCharset = 'UTF-8'
 

gradle/ide/eclipse/dot.settings/org.eclipse.jdt.core.prefs

@@ -1,5 +1,8 @@
 #Sun Sep 23 13:02:27 EDT 2012
 eclipse.preferences.version=1
+org.eclipse.jdt.core.compiler.codegen.targetPlatform=@eclipseJavaVersion@
+org.eclipse.jdt.core.compiler.compliance=@eclipseJavaVersion@
+org.eclipse.jdt.core.compiler.source=@eclipseJavaVersion@
 # Note: all compiler flags are defined in ECJ linter configuration
 @ecj-lint-config@
 org.eclipse.jdt.core.formatter.align_type_members_on_columns=false

gradle/validation/ecj-lint.gradle

@@ -60,6 +60,8 @@ allprojects {
         classpath = rootProject.configurations.ecjDeps
         mainClass = "org.eclipse.jdt.internal.compiler.batch.Main"
 
+        // DOCS: https://help.eclipse.org/latest/index.jsp?topic=%2Forg.eclipse.jdt.doc.user%2Ftasks%2Ftask-using_batch_compiler.htm
+
         // Don't emit any .class files.
         // Hack around "-d none" still emitting package-info.class
         // by running in a temporary directory.
@@ -68,6 +70,8 @@ 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 ]

gradle/validation/ecj-lint/ecj.javadocs.prefs

@@ -10,9 +10,7 @@ org.eclipse.jdt.core.compiler.annotation.nullable.secondary=
 org.eclipse.jdt.core.compiler.annotation.nullanalysis=disabled
 org.eclipse.jdt.core.compiler.codegen.inlineJsrBytecode=enabled
 org.eclipse.jdt.core.compiler.codegen.methodParameters=do not generate
-org.eclipse.jdt.core.compiler.codegen.targetPlatform=21
 org.eclipse.jdt.core.compiler.codegen.unusedLocal=preserve
-org.eclipse.jdt.core.compiler.compliance=21
 org.eclipse.jdt.core.compiler.debug.lineNumber=generate
 org.eclipse.jdt.core.compiler.debug.localVariable=generate
 org.eclipse.jdt.core.compiler.debug.sourceFile=generate
@@ -143,7 +141,6 @@ org.eclipse.jdt.core.compiler.problem.unusedTypeParameter=error
 org.eclipse.jdt.core.compiler.problem.unusedWarningToken=ignore
 org.eclipse.jdt.core.compiler.problem.varargsArgumentNeedCast=error
 org.eclipse.jdt.core.compiler.release=disabled
-org.eclipse.jdt.core.compiler.source=21
 org.eclipse.jdt.core.compiler.taskCaseSensitive=enabled
 org.eclipse.jdt.core.compiler.taskPriorities=HIGH
 org.eclipse.jdt.core.compiler.taskTags=nocommit