-
Notifications
You must be signed in to change notification settings - Fork 200
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
8289798: Update to use jtreg 7 #1595
Conversation
👋 Welcome back goetz! A progress list of the required criteria for merging this PR into |
This backport pull request has now been updated with issue from the original commit. |
GHA: broken. something wrong with locale. need to fix. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
jib-profiles.js line 209 looks like a typo. JTeport s/b JTreport.
Hey @GoeLin, we need to get jtreg updates in 17u :) Have you got any luck with figuring out the locale problem? Do you have time to work on it? Would you like me to try this separately? |
Failures are in this block of javadoc tests: jdk/javadoc/doclet/testSerialVersionUID/TestSerialVersionUID Not sure this is worth blocking the whole update, especially now this is blocking a backport (#1670) |
javadoc seems to be scanning inside the testng provided by jtreg:
|
I think backporting 8260265: UTF-8 by Default is the fix for the locale issues. If that is too much we would need to add |
That's a JEP implementation (https://bugs.openjdk.org/browse/JDK-8187041) so not suitable for backport. I guess
I guess we could try an explicit character set for these test cases. I'm unable to reproduce this locally, so it's hard to assess what's actually going on, and why it is even looking inside the testng JAR in the first place. |
#1672 sets the encoding and is working for me locally. |
This is superseded by #1712. |
I want to backport this on the path to jtreg 7.2.
I had to move the edit of lib-tests.m4 to toolchain.m4
All the others I resolved because we skipped the step to 6.1.
Progress
Issue
Reviewing
Using
git
Checkout this PR locally:
$ git fetch https://git.openjdk.org/jdk17u-dev.git pull/1595/head:pull/1595
$ git checkout pull/1595
Update a local copy of the PR:
$ git checkout pull/1595
$ git pull https://git.openjdk.org/jdk17u-dev.git pull/1595/head
Using Skara CLI tools
Checkout this PR locally:
$ git pr checkout 1595
View PR using the GUI difftool:
$ git pr show -t 1595
Using diff file
Download this PR as a diff file:
https://git.openjdk.org/jdk17u-dev/pull/1595.diff
Webrev
Link to Webrev Comment