-
Notifications
You must be signed in to change notification settings - Fork 706
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
Include the product name in the DTFJ JavaRuntime version #16941
Conversation
0043db9
to
bc283d3
Compare
debugtools/DDR_VM/src/com/ibm/j9ddr/vm29/tools/ddrinteractive/commands/CoreInfoCommand.java
Outdated
Show resolved
Hide resolved
debugtools/DDR_VM/src/com/ibm/j9ddr/vm29/view/dtfj/java/DTFJJavaRuntime.java
Outdated
Show resolved
Hide resolved
Addressed the review comments. |
debugtools/DDR_VM/src/com/ibm/j9ddr/vm29/tools/ddrinteractive/commands/CoreInfoCommand.java
Outdated
Show resolved
Hide resolved
debugtools/DDR_VM/src/com/ibm/j9ddr/vm29/view/dtfj/java/DTFJJavaRuntime.java
Outdated
Show resolved
Hide resolved
Jenkins compile alinux64 jdk17 |
This isn't quite done. Running
|
The line in
|
The jdmpview `info system` output includes the JavaRuntime version. Also include the product name in the !coreinfo output. Only OpenJ9 builds have a product name. Issue eclipse-openj9#16930 Signed-off-by: Peter Shipton <Peter_Shipton@ca.ibm.com>
Updated it and it's working now. |
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.
Build succeeded (https://openj9-jenkins.osuosl.org/view/Pull%20Requests/job/PullRequest-OpenJ9/3525/) and I verified locally that the latest change addresses the problem accessing older core files.
The jdmpview
info system
output includes the JavaRuntime version. Also include the product name in the !coreinfo output.Only OpenJ9 builds have a product name.
Closes #16930