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

Replaced JAVA_BIN to corresponding Test_JDK_HOME in Openj9 #6082

Merged
merged 1 commit into from
Jul 3, 2019

Conversation

LongyuZhang
Copy link
Contributor

  • Replaced JAVA_BIN to corresponding TEST_JDK_HOME to avoid ambiguity in openj9 test
  • modified native lib path for vmfarm
  • makefile, build_test, and settings.mk will be changed later along with JAVA_BIN in OpenJDK.
  • Merged codes which were used to handle different jdk versions but not needed now
  • Split codes which need to handle different jdk version due to removal of JAVA_BIN
    [ci skip]

Signed-off-by: Longyu [email protected]

- Replaced JAVA_BIN to corresponding TEST_JDK_HOME to avoid ambiguity in openj9 test
- modified native lib path for vmfarm
- makefile, build_test, and settings.mk will be changed later along with JAVA_BIN in OpenJDK.
- Merged codes which were used to handle different jdk versions but not needed now
- Split codes which need to handle different jdk version due to removal of JAVA_BIN
[ci skip]

Signed-off-by: Longyu <[email protected]>
@LongyuZhang
Copy link
Contributor Author

reviewers: @llxia @renfeiw

@llxia
Copy link
Contributor

llxia commented Jun 11, 2019

Fixed the issue mentioned in #5850

@llxia
Copy link
Contributor

llxia commented Jun 11, 2019

Jenkins test sanity,extended all jdk8

@llxia
Copy link
Contributor

llxia commented Jun 12, 2019

Jenkins test sanity,extended all jdk11

@llxia
Copy link
Contributor

llxia commented Jun 13, 2019

TestAttachErrorHandling_SE80_0 faiure is related to #3081

Copy link
Contributor

@llxia llxia left a comment

Choose a reason for hiding this comment

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

LGTM

@llxia llxia merged commit 0e894bb into eclipse-openj9:master Jul 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants