From 6819419f8cc97e461a74cefc29baa6d50d7d242f Mon Sep 17 00:00:00 2001
From: Marcono1234 <Marcono1234@users.noreply.github.com>
Date: Fri, 4 Jun 2021 02:31:14 +0200
Subject: [PATCH] Remove Maven toolchain requirement

---
 gson/pom.xml | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/gson/pom.xml b/gson/pom.xml
index b5b17a4a64..848cd4e552 100644
--- a/gson/pom.xml
+++ b/gson/pom.xml
@@ -25,11 +25,13 @@
         <artifactId>maven-surefire-plugin</artifactId>
         <version>3.0.0-M5</version>
         <configuration>
-          <jdkToolchain>
-            <!-- Use Java 11 for tests -->
-            <version>11</version>
-          </jdkToolchain>
           <!-- Deny illegal access, this is required for ReflectionAccessTest -->
+          <!-- Requires Java >= 9; Important: In case future Java versions 
+            don't support this flag anymore, don't remove it unless CI also runs with 
+            that Java version. Ideally would use toolchain to specify that this should 
+            run with e.g. Java 11, but Maven toolchain requirements (unlike Gradle ones) 
+            don't seem to be portable (every developer would have to set up toolchain 
+            configuration locally). -->
           <argLine>--illegal-access=deny</argLine>
         </configuration>
       </plugin>