-
Notifications
You must be signed in to change notification settings - Fork 738
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Checklist for Java new level bringup #17901
Comments
This could be unnecessary by adding a makefile under https://github.com/eclipse-openj9/openj9/tree/master/test/functional/cmdLineTests/jvmtitests similar to variables.mk by setting excludelist for jdk8 jvmtitests_excludes_80.xml and jdk11+ as general one jvmtitests_excludes.xml. As all jvmtitests_excludes_11+.xml seems exactly same and suppose it should be same for newer versions. And add the makefile to playlist |
Technically this could also be unnecessary if preferred. Checking if EXCLUDE_FILE:=$(TEST_ROOT)$(D)TestConfig$(D)resources$(D)excludes$(D)$(JCL_VERSION)exclude$(JDK_VERSION).txt exists or not. Only export if available or not empty string. Testng extended util openj9/test/Utils/src/org/openj9/test/util/IncludeExcludeTestAnnotationTransformer.java Lines 38 to 45 in 16964f3
|
Adoptium would remove that requirement for hotspot. adoptium/aqa-tests#5932 |
FYI @llxia |
Just for the record, the enhancement issue is opened at adoptium/aqa-tests#5890 |
openj9/jcl/jpp_configuration.xml
Line 189 in 0d64feb
jvmtitests excludes XML
file for Java new levelhttps://github.com/eclipse-openj9/openj9/blob/master/test/functional/cmdLineTests/jvmtitests/jvmtitests_excludes_22.xml
TestConfig latest exclude
file for Java new levelhttps://github.com/eclipse-openj9/openj9/blob/master/test/TestConfig/resources/excludes/latest_exclude_22.txt
openjdk/excludes/ProblemList_openjdk-openj9
file for Java new levelhttps://github.com/adoptium/aqa-tests/blob/master/openjdk/excludes/ProblemList_openjdk21-openj9.txt
The Next Java new level is Java 23.
The text was updated successfully, but these errors were encountered: