Skip to content
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

Fix the J9JDK_EXT_VERSION and remove obsolete JDK_MOD_VERSION #603

Merged
merged 1 commit into from
Aug 17, 2022

Conversation

pshipton
Copy link
Member

#532 modified the
J9JDK_EXT_VERSION to use JDK_UPDATE_VERSION instead of JDK_MOD_VERSION,
and COOKED_BUILD_NUMBER instead of JDK_FIX_VERSION. This obsoleted
JDK_MOD_VERSION which can be removed. Using COOKED_BUILD_NUMBER in the
J9JDK_EXT_VERSION is wrong, this is the IBM version for the extensions.
The J9JDK_EXT_VERSION should end with .0 typically, and be modified any
time there is a new extensions release for the same JDK_UPDATE_VERSION.

Signed-off-by: Peter Shipton Peter_Shipton@ca.ibm.com

ibmruntimes#532 modified the
J9JDK_EXT_VERSION to use JDK_UPDATE_VERSION instead of JDK_MOD_VERSION,
and COOKED_BUILD_NUMBER instead of JDK_FIX_VERSION. This obsoleted
JDK_MOD_VERSION which can be removed. Using COOKED_BUILD_NUMBER in the
J9JDK_EXT_VERSION is wrong, this is the IBM version for the extensions.
The J9JDK_EXT_VERSION should end with .0 typically, and be modified any
time there is a new extensions release for the same JDK_UPDATE_VERSION.

Signed-off-by: Peter Shipton <Peter_Shipton@ca.ibm.com>
pshipton added a commit to pshipton/openj9-openjdk-jdk8 that referenced this pull request Aug 17, 2022
Port of ibmruntimes#603 for
0.33.1, setting the JDK_FIX_VERSION to 1.

Signed-off-by: Peter Shipton <Peter_Shipton@ca.ibm.com>
@keithc-ca
Copy link
Member

jenkins test sanity zlinux jdk8

@keithc-ca keithc-ca merged commit 48806ae into ibmruntimes:openj9 Aug 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants