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

Build with OpenSSL v1.1.1 on AIX #124

Merged
merged 2 commits into from
Oct 9, 2018
Merged

Build with OpenSSL v1.1.1 on AIX #124

merged 2 commits into from
Oct 9, 2018

Conversation

mbvreddy
Copy link
Contributor

@mbvreddy mbvreddy commented Oct 8, 2018

This PR is for issue #123

#112 enabled OpenSSL support in openJ9-openjdk builds which also allow bundling of OpenSSL crypto library with JDK build. However build fails with --enable-openssl-bundling option as OpenSSL 1.1.1 changed the library name to align with the natural way of handling shared library on AIX. Corresponding openssl PR for reference - openssl/openssl#6487

As a result, we now have to link and bundle libcrypto.a instead of libcrypto.so.1.1.

In this fix,

  • we copy libcrypto.a to jre/lib/ppc64 directory if built with OpenSSL v1.1.1 and libcrypto.so to jre/lib/ppc64 if built with OpenSSL 1.1.0x
  • remove linking against ssl on all platforms as our implementation use only crypto library

Signed-off-by: Bhaktavatsal Reddy bhamaram@in.ibm.com

Signed-off-by: Bhaktavatsal Reddy <bhamaram@in.ibm.com>
@mbvreddy mbvreddy changed the title Build with OpenSSL v1.1.1 on AIX WIP - Build with OpenSSL v1.1.1 on AIX Oct 8, 2018
…h OpenSSL v1.1.0

Signed-off-by: Bhaktavatsal Reddy <bhamaram@in.ibm.com>
@mbvreddy mbvreddy changed the title WIP - Build with OpenSSL v1.1.1 on AIX Build with OpenSSL v1.1.1 on AIX Oct 9, 2018
@andrew-m-leonard
Copy link
Contributor

jenkins compile aix

@andrew-m-leonard andrew-m-leonard merged commit 8885d54 into ibmruntimes:openj9 Oct 9, 2018
pshipton added a commit to pshipton/openj9 that referenced this pull request Oct 9, 2018
Reverts part of eclipse-openj9#3171 to enable AIX.
Previous problem was fixed by the merge of
ibmruntimes/openj9-openjdk-jdk8#124

Signed-off-by: Peter Shipton <Peter_Shipton@ca.ibm.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants