paulk-asert commented on code in PR #1771:
URL: https://github.com/apache/groovy/pull/1771#discussion_r956682803
##########
buildSrc/src/main/groovy/org.apache.groovy-tested.gradle:
##########
@@ -32,12 +32,18 @@ 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"]
Review Comment:
And from memory, a few tests in groovy-templates. Plus legal usage of clone
seems broken too:
```
int[] nums = [1, 2, 3]
nums.clone()
```
==>
BUG! Unknown transformation for argument [I@503a20ea at position 0 with
class [I for parameter of type class
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
