Gradle: Configure compiler options via build-logic

build-logic/base/src/main/groovy/org/spockframework/gradle/SpockBasePlugin.groovy

@@ -19,11 +19,43 @@ package org.spockframework.gradle
 import groovy.transform.CompileStatic
 import org.gradle.api.Plugin
 import org.gradle.api.Project
+import org.gradle.api.plugins.JavaPlugin
+import org.gradle.api.tasks.compile.GroovyCompile
+import org.gradle.api.tasks.compile.JavaCompile
 import org.gradle.api.tasks.testing.Test
+import org.gradle.jvm.toolchain.JavaLanguageVersion
+import org.gradle.jvm.toolchain.JavaToolchainService
+import org.jetbrains.annotations.VisibleForTesting
 
 @CompileStatic
 class SpockBasePlugin implements Plugin<Project> {
+
+  @VisibleForTesting
+  public static final JavaLanguageVersion COMPILER_VERSION = JavaLanguageVersion.of(8)
+
   void apply(Project project) {
+    compileTasks(project)
+    testTasks(project)
+  }
+
+  private static void compileTasks(Project project) {
+    project.with {
+      def javaToolchains = extensions.getByType(JavaToolchainService)
+      tasks.withType(JavaCompile).configureEach { comp ->
+        if (comp.name == JavaPlugin.COMPILE_JAVA_TASK_NAME) {
+          comp.javaCompiler.set(javaToolchains.compilerFor {
+            it.languageVersion.set(COMPILER_VERSION)
+          })
+        }
+        comp.options.encoding = 'UTF-8'
+      }
+      tasks.withType(GroovyCompile).configureEach {
+        it.options.encoding = 'UTF-8'
+      }
+    }
+  }
+
+  private static void testTasks(Project project) {
     project.tasks.withType(Test).configureEach { task ->
       def taskName = task.name.capitalize()
       File configFile = project.file("Spock${taskName}Config.groovy")

build-logic/base/src/test/groovy/org/spockframework/gradle/SpockBasePluginSpec.groovy

@@ -0,0 +1,32 @@
+package org.spockframework.gradle
+
+import org.gradle.api.Project
+import org.gradle.api.plugins.JavaPlugin
+import org.gradle.api.tasks.compile.JavaCompile
+import org.gradle.testfixtures.ProjectBuilder
+import spock.lang.Specification
+
+class SpockBasePluginSpec extends Specification {
+
+  def 'Compile settings are configured'() {
+    setup:
+    def project = createProject()
+
+    when:
+    def compileJavaTasks = project.tasks.withType(JavaCompile)
+    def compileJava = project.tasks.getByName(JavaPlugin.COMPILE_JAVA_TASK_NAME) as JavaCompile
+
+    then:
+    compileJavaTasks.every { it.options.encoding == "UTF-8" }
+    compileJava.javaCompiler.get().metadata.languageVersion == SpockBasePlugin.COMPILER_VERSION
+  }
+
+  private static Project createProject() {
+    def result = ProjectBuilder.builder()
+      .build()
+    result.plugins.apply("java-library")
+    result.plugins.apply("groovy")
+    result.plugins.apply(SpockBasePlugin)
+    return result
+  }
+}

build.gradle

@@ -102,25 +102,14 @@ subprojects {
   apply plugin: "java-library"
   apply plugin: "groovy"
   apply plugin: "jacoco"
+  apply plugin: "org.spockframework.base"
 
   java {
     toolchain {
       languageVersion = JavaLanguageVersion.of(javaVersion)
     }
   }
 
-  tasks.withType(JavaCompile).configureEach {
-    if (it.name == 'compileJava') {
-      javaCompiler = javaToolchains.compilerFor {
-        languageVersion = JavaLanguageVersion.of(8)
-      }
-    }
-    options.encoding = 'UTF-8'
-  }
-  tasks.withType(GroovyCompile).configureEach {
-    options.encoding = 'UTF-8'
-  }
-
   sourceSets.all { ss ->
     for (gv in variants.findAll { variant <= it }) {
       java {

spock-core/core.gradle

@@ -96,7 +96,7 @@ tasks.named("processResources") {
 tasks.register("coreConsole", JavaExec) {
   description = 'Start a groovy Console with Spock Core Classpath, useful for AST-Inspection'
   mainClass = variant == 2.5 ? "groovy.ui.Console" : "groovy.console.ui.Console"
-  classpath(sourceSets.named("main").map {it.runtimeClasspath }, configurations.named("coreConsoleRuntime"))
+  classpath(sourceSets.named("main").map { it.runtimeClasspath }, configurations.named("coreConsoleRuntime"))
   workingDir = file('build/console')
   ignoreExitValue true
   args file('CoreConsole.groovy').absolutePath
@@ -119,7 +119,7 @@ def osgiProperties = tasks.register('osgiProperties', WriteProperties) {
 // that its metadata is valid. If the metadata is invalid this task will
 // fail.
 def verifyOSGi = tasks.register('verifyOSGi', Resolve) {
-  getBndrun().fileProvider(osgiProperties.map { it.outputFile })
+  getBndrun().set(osgiProperties.flatMap { it.destinationFile })
   getOutputBndrun().set(layout.getBuildDirectory().file("resolvedOSGiProperties.bndrun"))
   reportOptional = false
   // By default bnd will use jars found in:

spock-specs/specs.gradle

@@ -1,9 +1,5 @@
 import org.spockframework.gradle.JacocoJavaagentProvider
 
-plugins {
-  id 'org.spockframework.base'
-}
-
 ext.displayName = "Spock Framework - Specs for Core Module"
 
 description = "Spock specifications for the Core Module. Yes, we eat our own dog food."

spock-testkit/testkit.gradle

@@ -1,7 +1,3 @@
-plugins {
-  id 'org.spockframework.base'
-}
-
 ext.displayName = "Spock Framework - Temp Specs for Core Module"
 
 //configurations {