-
Notifications
You must be signed in to change notification settings - Fork 115
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
Copy files to JDK explicitly #210
Conversation
jenkins test sanity zlinux |
jenkins test all xlinux,plinux,win,aix |
jenkins test extended zlinux |
@llxia @sophia-guo please see the extended test AttachAPI failures. Seems we are not quite ready to remove the test libraries from the jdk8 builds. |
Yes, for JDK8 when enable native tests only executables related Variables are updated. Shared libraries related variables are keep the same. I will update that. |
jenkins test extended zlinux jdk8 |
@sophia-guo 's change is in. |
@keithc-ca reminder this needs to be rebased. |
Other shared libraries link to the redirector libjvm.so: it must be present in the lib/server directory which is included in the library search path. The library in the vm directory is the real VM, not a redirector. The adjustments to LDFLAGS_JDKLIB in custom-hook.m4 weren't correct and are no longer helpful. Also remove unnecessary use of () for scripts. Signed-off-by: Keith W. Campbell <[email protected]>
Signed-off-by: Keith W. Campbell <[email protected]>
Signed-off-by: Keith W. Campbell <[email protected]>
I rebased, but I'm not sure it was really necessary. The failure I see in https://ci.eclipse.org/openj9/job/PullRequest-Extended-JDK8-linux_390-64_cmprssptrs-OpenJDK8/8/console seems to have nothing to do with this:
The copyright failures seem to be because the check doesn't look far enough into the (generated) files. |
Jenkins test all aix,plinux,win,xlinux |
jenkins test extended win |
Explicitly copy files to images like was done in ibmruntimes/openj9-openjdk-jdk11#16.
The effect of this change is to add this file:
and omit these (which can be found in
images/test/openj9
):Fixes #17
Fixes eclipse-openj9/openj9#805