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

Upgrade to asm-7.2.jar in tests #7553

Merged
merged 1 commit into from
Oct 23, 2019
Merged

Conversation

llxia
Copy link
Contributor

@llxia llxia commented Oct 22, 2019

Some tests are using asm-7.0-beta.jar and some are using asm-7.2.jar.
Remove asm-7.0-beta.jar and upgrade tests to use asm-7.2.jar, so we have one
less dependent jar in tests

[ci skip]

Signed-off-by: lanxia [email protected]

@llxia llxia changed the title Upgrade to asm-7.0-beta.jar in tests Upgrade to asm-7.2.jar in tests Oct 22, 2019
Some tests are using asm-7.0-beta.jar and some are using asm-7.2.jar
Remove asm-7.0-beta.jar and upgrade tests to asm-7.2.jar, so we have one
less dependent jar in tests

[ci skip]

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

llxia commented Oct 22, 2019

jenkins test extended all jdk8,jdk11,jdk13

@llxia
Copy link
Contributor Author

llxia commented Oct 23, 2019

All tests passed except JDK13 on AIX. It can be ignored as JDK13 on AIX is not ready #7417 (comment)

@llxia llxia marked this pull request as ready for review October 23, 2019 15:16
@llxia llxia requested a review from smlambert October 23, 2019 15:16
Copy link
Contributor

@smlambert smlambert left a comment

Choose a reason for hiding this comment

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

LGTM

@smlambert smlambert merged commit 9a05e0a into eclipse-openj9:master Oct 23, 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