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

[Bug] Alibaba Dragonwell version is not correct. #30

Closed
yfxhust opened this issue May 23, 2019 · 1 comment
Closed

[Bug] Alibaba Dragonwell version is not correct. #30

yfxhust opened this issue May 23, 2019 · 1 comment

Comments

@yfxhust
Copy link
Contributor

yfxhust commented May 23, 2019

Description
With java -version, dragonwell version is not correct.

Steps to Reproduce

  1. execute sh make.sh to build dragonwell8
  2. Execute 'java -version' with the built jdk
  3. You will got the following output
openjdk version "1.8.0_202"
OpenJDK Runtime Environment (Dragonwell 8.0-preview) (build 1.8.0_202-b37)
OpenJDK 64-Bit Server VM (Dragonwell 8.0-preview) (build 25.202-b37, mixed mode)

Expected behavior
The java -version should reflect the distro build and upstream build. So the expected output should be

openjdk version "1.8.0_202"
OpenJDK Runtime Environment (Dragonwell 8.0.0-b00) (build 1.8.0_202-b01)
OpenJDK 64-Bit Server VM (Dragonwell 8.0.0-b00) (build 25.202-b01, mixed mode)

JDK version
Any

Execution environment
Any

yfxhust added a commit to yfxhust/dragonwell8 that referenced this issue May 23, 2019
Summary:
Update dragonwell version to 8.0.0

Test Plan: make.sh saint check.

Reviewed-by: D-D-H, mmyxym, luchsh

Issue: dragonwell-project#30
yfxhust added a commit to yfxhust/dragonwell8 that referenced this issue May 24, 2019
Summary:
Update dragonwell version to 8.0.0

Test Plan: make.sh saint check.

Reviewed-by: D-D-H, mmyxym, luchsh

Issue: dragonwell-project#30
yfxhust added a commit that referenced this issue May 27, 2019
Summary:
Update dragonwell version to 8.0.0

Test Plan: make.sh saint check.

Reviewed-by: D-D-H, mmyxym, luchsh

Issue: #30
@yfxhust
Copy link
Contributor Author

yfxhust commented May 30, 2019

After applying patch, it is correct now.

@yfxhust yfxhust closed this as completed May 30, 2019
yfxhust added a commit that referenced this issue Jun 22, 2019
Summary:
Update dragonwell version to 8.0.0

Test Plan: make.sh saint check.

Reviewed-by: D-D-H, mmyxym, luchsh

Issue: #30
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

No branches or pull requests

1 participant