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

Link redirector with libdl.so on non-Windows platforms #5148

Merged
merged 1 commit into from
Mar 19, 2019

Conversation

keithc-ca
Copy link
Contributor

Avoids errors linking to (redirector) libjvm.so alone.

Fixes: #5047

* avoids errors linking to (redirector) libjvm.so alone

Fixes: eclipse-openj9#5047

Signed-off-by: Keith W. Campbell <keithc@ca.ibm.com>
Copy link
Member

@DanHeidinga DanHeidinga left a comment

Choose a reason for hiding this comment

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

Looks reasonable.

fyi @dnakamura as cmake may need a similar change

@DanHeidinga
Copy link
Member

Jenkins test sanity plinux jdk8

@DanHeidinga
Copy link
Member

Jenkins compile win,zlinux,aix jdk11

@DanHeidinga
Copy link
Member

Jenkins compile osx jdk12

@DanHeidinga DanHeidinga self-assigned this Mar 19, 2019
@keithc-ca
Copy link
Contributor Author

I had forgotten about cmake, but it appears it already links with -ldl.

@DanHeidinga DanHeidinga merged commit 7a369a8 into eclipse-openj9:master Mar 19, 2019
@keithc-ca keithc-ca deleted the link-redirector branch March 19, 2019 21:11
keithc-ca added a commit to keithc-ca/openj9 that referenced this pull request Mar 20, 2019
Refines: eclipse-openj9#5148
Signed-off-by: Keith W. Campbell <keithc@ca.ibm.com>
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.

3 participants