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

[Misc] Update dragonwell version to 8.3.3 #93

Merged
merged 1 commit into from
Apr 6, 2020
Merged

[Misc] Update dragonwell version to 8.3.3 #93

merged 1 commit into from
Apr 6, 2020

Conversation

luchsh
Copy link
Contributor

@luchsh luchsh commented Apr 4, 2020

Summary:
Update dragonwell version to 8.3.3 and removed DRAGONWELL_JDK_BUILD_NUMBER

Test Plan:
make.sh saint check

Reviewed-by: yuleil, kuaiwei

Issue: #92

Summary:
Update dragonwell version to 8.3.3 and removed DRAGONWELL_JDK_BUILD_NUMBER

Test Plan:
make.sh saint check

Reviewed-by: yuleil, kuaiwei

Issue: #92
@luchsh luchsh requested review from kuaiwei and yuleil April 4, 2020 08:11
@luchsh
Copy link
Contributor Author

luchsh commented Apr 4, 2020

Currently there is a unique 'GA' tag for each update release of OpenJDK, such as 'jdk8u242-ga', so DRAGONWELL_JDK_BUILD_NUMBER is always '-ga' and not needed any more.
So I consolidate the version pattern to the same way of AJDK in this PR.

Copy link
Collaborator

@kuaiwei kuaiwei left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please update copyright to 2020

DRAGONWELL_JDK_UPDATE_VERSION=232
DRAGONWELL_JDK_BUILD_NUMBER=b09
DRAGONWELL_VERSION=8.3.3
DRAGONWELL_JDK_UPDATE_VERSION=242
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't find use of DRAGONWELL_JDK_UPDATE_VERSION in make.sh, do we need this variable?

Copy link
Contributor Author

@luchsh luchsh Apr 5, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is used by ./common/autoconf/jdk-options.m4, to enable creation of Dragonwell JDK builds from the standard configure && make scripts without involving make.sh.

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.

3 participants