Use "release" option to fix Java 8 binary compatibility

marianobarrios · Jul 6, 2024 · 81b9130 · 81b9130
1 parent 8263cd5
commit 81b9130
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 4 deletions.
diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml
@@ -14,6 +14,7 @@ jobs:
       - uses: actions/setup-java@v4
         with:
           distribution: 'temurin'
+          # The target JDK is used to build, the latest one to run Gradle itself
           java-version: |
             ${{ matrix.java-version }}
             21

diff --git a/build.gradle b/build.gradle
@@ -9,22 +9,27 @@ plugins {
 var javaToolchain = System.getenv('JAVA_TOOLCHAIN')
 
 java {
+    // If running from CI, the tool chain will be present
     if (javaToolchain != null) {
         toolchain {
             languageVersion = JavaLanguageVersion.of(javaToolchain)
         }
     }
 }
 
+println(javaToolchain)
+
 compileJava {
-    sourceCompatibility = '8'
-    targetCompatibility = '8'
+    if (javaToolchain != "8") {
+        options.release = 8
+    }
     options.compilerArgs.add('-Xlint')
 }
 
 compileTestJava {
-    sourceCompatibility = '8'
-    targetCompatibility = '8'
+    if (javaToolchain != "8") {
+        options.release = 8
+    }
     options.compilerArgs.add('-Xlint')
 }