daniellansun commented on code in PR #1771:
URL: https://github.com/apache/groovy/pull/1771#discussion_r956701779
##########
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
I added a test case for the above code, fails with gradle 7.5.1
https://github.com/apache/groovy/commit/ce15eaef34d7d77d0186dd82f7132e6644ad8761
--
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]
