Use toolchains to configure build and test Java version

Author: mrotteveel

.github/workflows/run-tests.yml

@@ -8,6 +8,25 @@ permissions:
 
 jobs:
   build:
+    strategy:
+      matrix:
+        java:
+          - version: 17
+            javaInstall: 17
+            buildProfile: 17
+            testProfile: 17
+          - version: 17-test-21
+            javaInstall: |
+              17
+              21
+            buildProfile: 17
+            testProfile: 21
+          - version: 17-test-25
+            javaInstall: |
+              17
+              25
+            buildProfile: 17
+            testProfile: 25
     runs-on: ubuntu-latest
     steps:
       - name: Login to Docker Hub
@@ -21,22 +40,22 @@ jobs:
           firebird_root_password: 'masterkey'
           firebird_conf: 'WireCrypt=Enabled,AuthServer=Srp256;Srp;Legacy_Auth;Srp224;Srp384;Srp512,UserManager=Srp;Legacy_UserManager'
       - uses: actions/checkout@v6
-      - name: Set up Java 17
+      - name: Set up Java ${{ matrix.java.version }}
         uses: actions/setup-java@v5
         with:
-          java-version: '17'
+          java-version: ${{ matrix.java.javaInstall }}
           distribution: 'temurin'
       - name: Setup Gradle
         uses: gradle/actions/setup-gradle@0723195856401067f7a2779048b490ace7a47d7c
         with:
           cache-read-only: ${{ github.ref != 'refs/heads/master' }}
       - name: Build with Gradle
-        run: ./gradlew test -Ptest.dbondocker=true -Ptest.db.dir=/var/lib/firebird/data
+        run: ./gradlew test -PbuildProfile=${{ matrix.java.buildProfile }} -PtestProfile=${{ matrix.java.testProfile }} -Ptest.dbondocker=true -Ptest.db.dir=/var/lib/firebird/data
       - name: Store Report Artifact
         uses: actions/upload-artifact@v7
         if: always()
         with:
-          name: report-artifacts
+          name: report-artifacts-${{ matrix.java.version }}
           path: build/reports
           compression-level: 9
           retention-days: 7

build-properties.gradle

@@ -24,10 +24,24 @@ def buildTime = LocalDateTime.now()
 ext.'build.id' = buildTime.format('yyyyMMddHHmm')
 ext.YEAR = buildTime.format('yyyy')
 
-// Build profile
-ext.'maven.jdkversion' = 'java17'
+// Build profile (target Java version)
+def buildProfileProp = findProperty('buildProfile') as String
+def buildMatcher = buildProfileProp =~ /^(?:java)?(\d+)$/
+ext.buildProfile = buildMatcher ? buildMatcher.group(1) as Integer : 17
+println "Build profile: Java ${ext.buildProfile}"
+if (ext.buildProfile != 17) {
+    println "WARNING: Java target version ${ext.buildProfile} (buildProfile) is not a baseline version and will not be published to Maven."
+}
+
+// As we only publish the Java 17 version, we don't do further mapping for higher JDBC versions
 ext.'specification.version' = '4.3'
 
+// Test profile (test Java version)
+def testProfileProp = findProperty('testProfile') as String
+def testMatcher =  testProfileProp =~ /^(?:java)?(\d+)$/
+ext.testProfile = testMatcher ? testMatcher.group(1) as Integer : ext.buildProfile
+println "Test profile: Java ${ext.testProfile}"
+
 ext.'version.simple' = "${project.'version.major'}.${project.'version.minor'}.${project.'version.revision'}".toString()
 ext.'version.maven' = "${project.'version.simple'}${project.'version.tag'}".toString()
 

build.gradle

@@ -6,6 +6,21 @@ import org.asciidoctor.gradle.jvm.AsciidoctorTask
 /*
   Gradle build script for Jaybird - Firebird JDBC driver.
 
+  Define target Java version:
+  - -PbuildProfile=17              : Java 17 (supported)
+    or not specified
+  - -PbuildProfile=NN              : Java NN (not publishable)
+    with NN > 17
+
+  Build profiles other than 17 will not be published to Maven.
+
+  Define (other) Java version to test:
+  - -PtestProfile=NN with NN >= 17
+
+  If not specified, the test profile is the same as buildProfile.
+
+  For historic reasons, both buildProfile and testProfile also accept javaNN.
+
   Uploading archives:
 
   publish -PcredentialsPassphrase=<credentials password>
@@ -29,9 +44,22 @@ version = project.'version.maven'
 allprojects {
     tasks.withType(JavaCompile).configureEach {
         options.encoding = 'UTF-8'
+        int buildProfile = project.buildProfile
+        options.release = buildProfile
+        javaCompiler = javaToolchains.compilerFor {
+            languageVersion = JavaLanguageVersion.of((int) buildProfile)
+        }
     }
     tasks.withType(Test).configureEach {
         systemProperty 'file.encoding', 'UTF-8'
+        javaLauncher = javaToolchains.launcherFor {
+            languageVersion = JavaLanguageVersion.of((int) project.testProfile)
+        }
+    }
+    tasks.withType(Javadoc).configureEach {
+        javadocTool = javaToolchains.javadocToolFor {
+            languageVersion = JavaLanguageVersion.of((int) project.testProfile)
+        }
     }
 }
 
@@ -40,8 +68,9 @@ base {
 }
 
 java {
-    sourceCompatibility = JavaVersion.VERSION_17
-    targetCompatibility = JavaVersion.VERSION_17
+    def javaVersion = JavaLanguageVersion.of((int) project.buildProfile)
+    sourceCompatibility = javaVersion
+    targetCompatibility = javaVersion
     withJavadocJar()
     withSourcesJar()
 }
@@ -161,13 +190,12 @@ tasks.named('assemble') {
 jar {
     manifest {
         attributes(
-                'Created-By': "${System.getProperty('java.vm.version')} (${System.getProperty('java.vm.vendor')})",
                 'Specification-Title': project.'specification.title',
                 'Specification-Version': project.'specification.version',
                 'Specification-Vendor': project.'specification.vendor',
                 'Implementation-Title': project.'implementation.title',
                 'Implementation-Url': project.'implementation.url',
-                'Implementation-Version': "$project.version (build: variant=$project.mavenName tag=${project.'version.svntag'} date=${project.'build.id'})",
+                'Implementation-Version': "$project.version (build: variant=$project.mavenName tag=${project.'version.svntag'} date=${project.'build.id'} profile=${project.buildProfile})",
                 'Implementation-Vendor': project.'implementation.vendor',
                 'Implementation-Vendor-Id': project.'implementation.vendor.id',
                 'Bundle-License': 'LGPL-2.1-or-later OR LGPL-2.1-or-later AND BSD-3-Clause',

devdoc/build-documentation.md

@@ -10,15 +10,23 @@ please contribute using a pull request, or ask on [firebird-java](https://groups
 ## Running the build
 
 The build uses Gradle wrapper, which is included in the repository. The current
-minimum version is Java 17.
+minimum version is Java 17. The build target (buildProfile) defaults to Java 17,
+independent of the Java version running Gradle.
+
+The target Java version is determined by passing `-PbuildProfile=NN`, where `NN`
+is the desired Java version. The default is `17`. Versions other than `17` are
+not published to Maven.
+
+By default, tests are run with that `buildProfile` version. This can be
+overridden with `-PtestProfile=NN`. For historic reasons, `javaNN` is also
+accepted by both properties.
 
 To run with a specific Java version, we suggest creating a Java-specific launch
 script. For example, for Windows create a batch file with:
 
 ```
 @echo off
-set JAVA_HOME=C:\Program Files\Java\jdk-19
-call gradlew.bat %*
+call gradlew.bat -PbuildProfile=21 %*
 ```
 
 ### Default build

publish.gradle

@@ -186,6 +186,9 @@ publishing {
 
 allprojects {
     tasks.withType(PublishToMavenRepository).each {
+        it.onlyIf('Only publish Java 17 builds') {
+            project.buildProfile == 17
+        }
         it.doFirst {
             if (findProperty('centralUsername') == null || findProperty('centralPassword') == null) {
                 throw new RuntimeException('No credentials for publishing, make sure to specify the properties ' +