[groovy] 01/02: GROOVY-10730: Remove the jdk16+ add-opens jvm args

[su...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

sunlan pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/groovy.git

commit f3913253998953532c2e6cc99886ca12446496ad
Author: Daniel Sun <su...@apache.org>
AuthorDate: Sun Aug 28 22:24:10 2022 +0800

    GROOVY-10730: Remove the jdk16+ add-opens jvm args
---
 buildSrc/src/main/groovy/org.apache.groovy-tested.gradle | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/buildSrc/src/main/groovy/org.apache.groovy-tested.gradle b/buildSrc/src/main/groovy/org.apache.groovy-tested.gradle
index dfa7479e60..19ac5e7492 100644
--- a/buildSrc/src/main/groovy/org.apache.groovy-tested.gradle
+++ b/buildSrc/src/main/groovy/org.apache.groovy-tested.gradle
@@ -32,18 +32,12 @@ sourceSets {
     }
 }
 
-// TODO remove the jdk16+ add-opens jvm args once offending code/tests are fixed
-// https://docs.gradle.org/7.5/userguide/upgrading_version_7.html#removes_implicit_add_opens_for_test_workers
 tasks.withType(Test).configureEach {
     def fs = objects.newInstance(TestServices).fileSystemOperations
     def grapeDirectory = new File(temporaryDir, "grape")
     def jdk8 = ['-XX:+UseConcMarkSweepGC']
     def jdk9 = ['-Djava.locale.providers=COMPAT,SPI']
 //        def jdk9 = ['-Djava.locale.providers=COMPAT,SPI', '--illegal-access=debug']
-    if (JavaVersion.current().isCompatibleWith(JavaVersion.VERSION_16)) {
-        jdk9 += ["--add-opens=java.base/java.lang=ALL-UNNAMED",
-                 "--add-opens=java.base/java.util=ALL-UNNAMED"]
-    }
     def common = ['-ea', "-Xms${groovyJUnit_ms}", "-Xmx${groovyJUnit_mx}", "-Duser.language=en"]
     if (JavaVersion.current().isJava9Compatible()) {
         jvmArgs(*common, *jdk9)