adoptium
Build the tests with release=8 whenever it is possible
Fixes https://github.com/eclipse-openj9/openj9/issues/19888
The tests have to be built with release=8 in order to avoid calling the new overridden methods with covariant return types in java.nio.ByteBuffer/CharBuffer/LongBuffer, like position(int), rewind, flip... The only source file that has to be built with source=1.8 and target=1.8 is the gnu/testlet/java/lang/Class/classInfo/getDeclaredMethod.java, because it requires sun.reflect.annotation.AnnotationType that is not visible using release=8 from jdk9+ The specifying of release="8", source="1.8" and target="1.8" allows ant 1.10.x to use either release if supported or the source/target if release is not supported.
Thanks @fridrich for providing the fix. Could you try to sign ECA? https://github.com/adoptium/aqa-tests/blob/master/Contributing.md#eclipse-contributor-agreement
The process should be simple, but unfortunately, we cannot proceed with the change without a signed ECA.
FYI the comment which I'll copy here.
fridrich commented Sep 23, 2024
So, that pull request basically fixes the build of the mauve.jar, so that it works with the java 8. It is hanging in eclipsefdn/eca. I am not sure whether we (SUSE) have ever signed something like this. I also don't care to be credited, so anybody who has the right paperwork on file can submit that patch. I transfer automatically the copyright to anybody brave enough to get it integrated.
At the end, I signed it in my personal capacity. So it should be good I think.
Quick update: We encountered other issues while upgrading to JDK 17 for compilation for all dependent jars (though mauve.jar is fine). This PR can't be merged until those issues are resolved.