Clean up gradle-profiler build set up

- Reduce number of java toolchains used from 3 to 1 (just need JDK 17
  now)
- Move to a shared place for setting java source and target version to 8

Signed-off-by: Aurimas Liutikas <[email protected]>

Author: liutikas

build.gradle.kts

@@ -107,7 +107,7 @@ tasks.test {
             .getOrElse(JavaVersion.current().majorVersion)
         languageVersion.set(JavaLanguageVersion.of(javaVersion))
         providers.gradleProperty("testJavaVendor").map {
-            when (it.lowercase()) {
+            when (it.lowercase(Locale.US)) {
                 "oracle" -> vendor.set(JvmVendorSpec.ORACLE)
                 "openjdk" -> vendor.set(JvmVendorSpec.ADOPTIUM)
             }
@@ -144,7 +144,7 @@ val testReports = mapOf(
     "testHtmlReportWithOps" to "example-with-build-operations",
     "testHtmlReportRegression" to "example-regression"
 )
-testReports.forEach { taskName, fileName ->
+testReports.forEach { (taskName, fileName) ->
     tasks.register<ProcessResources>(taskName) {
         val dataFile = file("src/test/resources/org/gradle/profiler/report/${fileName}.json")
         inputs.file(dataFile)

buildSrc/src/main/kotlin/profiler.java-library.gradle.kts

@@ -18,10 +18,15 @@ repositories {
 
 java {
     toolchain {
-        languageVersion.set(JavaLanguageVersion.of(8))
+        languageVersion.set(JavaLanguageVersion.of(17))
     }
 }
 
+tasks.withType<AbstractCompile>().configureEach {
+    targetCompatibility = "8"
+    sourceCompatibility = "8"
+}
+
 tasks.withType<Test>().configureEach {
     // Add OS as inputs since tests on different OS may behave differently https://github.com/gradle/gradle-private/issues/2831
     // the version currently differs between our dev infrastructure, so we only track the name and the architecture

settings.gradle.kts

@@ -20,10 +20,10 @@ rootProject.children.forEach {
 }
 
 /**
- * Intellij-gradle-plugin requires Java 11.
+ * Building gradle-profile requires JDK 17.
  */
 fun checkIfCurrentJavaIsCompatible() {
-    if (!JavaVersion.current().isCompatibleWith(JavaVersion.VERSION_11)) {
-        throw GradleException("This project should be run with Java 11 or later, but it was run with Java ${JavaVersion.current()}.")
+    if (!JavaVersion.current().isCompatibleWith(JavaVersion.VERSION_17)) {
+        throw GradleException("This project should be build with JDK 17, but it was run with Java ${JavaVersion.current()}.")
     }
 }

subprojects/build-operations/build.gradle.kts

@@ -5,14 +5,3 @@ plugins {
 dependencies {
     api(gradleApi())
 }
-
-java {
-    toolchain {
-        languageVersion.set(JavaLanguageVersion.of(11))
-    }
-}
-
-tasks.withType<AbstractCompile>().configureEach {
-    targetCompatibility = "8"
-    sourceCompatibility = "8"
-}

subprojects/studio-agent/build.gradle.kts

@@ -2,12 +2,6 @@ plugins {
     id("profiler.embedded-library")
 }
 
-java {
-    toolchain {
-        languageVersion.set(JavaLanguageVersion.of(11))
-    }
-}
-
 dependencies {
     implementation("org.ow2.asm:asm:9.2")
     implementation(project(":client-protocol"))

subprojects/studio-plugin/build.gradle.kts

@@ -37,10 +37,15 @@ project.configurations
 
 java {
     toolchain {
-        languageVersion.set(JavaLanguageVersion.of(11))
+        languageVersion.set(JavaLanguageVersion.of(17))
     }
 }
 
+tasks.withType<AbstractCompile>().configureEach {
+    targetCompatibility = "11"
+    sourceCompatibility = "11"
+}
+
 tasks.test {
     useJUnitPlatform()
     // Disable IntelliJ file system access check for tests: having this check enabled can fail