-
-
Notifications
You must be signed in to change notification settings - Fork 317
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
Implement MultiJVM tck in one node #4372
Conversation
a) Test this PR with single-node solution using
b) Test this PR with multi-node solution using AIX : Grinder_JCK/1123. |
c5be279
to
451c7cb
Compare
3809d54
to
c35c20d
Compare
jck/jck.mk
Outdated
else ifeq ($(ARCH),s390x) | ||
USE_MULTINODE = true | ||
endif | ||
endif |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should be able to use $(SPEC)
. See https://github.com/adoptium/aqa-tests/blob/master/openjdk/openjdk.mk#L110 for example.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's simpler! Updated. Tested with AIX: Grinder_JCK/1125, xLinux: Grinder_JCK/1126.
4956466
to
ec0807f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @Mesbah-Alam
Signed-off-by: Mesbah Alam <Mesbah_Alam@ca.ibm.com>
ec0807f
to
75219ee
Compare
Implements backlog/issues/1035
Signed-off-by: Mesbah Alam Mesbah_Alam@ca.ibm.com