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

Enable openssl within nightly and pr builds #3158

Merged
merged 1 commit into from
Oct 5, 2018

Conversation

jdekonin
Copy link
Contributor

@jdekonin jdekonin commented Oct 5, 2018

fyi @pshipton

@jdekonin jdekonin force-pushed the master branch 2 times, most recently from 33a2902 to bcb6893 Compare October 5, 2018 14:48
@@ -279,11 +287,7 @@ linux_x86-64:
11: 'hw.arch.x86 && sw.os.ubuntu'
next: 'hw.arch.x86 && sw.os.ubuntu'
extra_configure_options:
8: '--with-noncompressedrefs'
9: '--with-noncompressedrefs'
Copy link
Member

Choose a reason for hiding this comment

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

Did you mean to remove these?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No I did not. Restored.

@@ -81,7 +81,7 @@ def get_sources() {
checkout_pullrequest()
} else {
sh "git checkout ${OPENJDK_SHA}"
sh "bash ./get_source.sh ${OPENJ9_REPO_OPTION} ${OPENJ9_BRANCH_OPTION} ${OPENJ9_SHA_OPTION} ${OMR_REPO_OPTION} ${OMR_BRANCH_OPTION} ${OMR_SHA_OPTION}"
sh "bash ./get_source.sh $EXTRA_GETSOURCE_OPTIONS ${OPENJ9_REPO_OPTION} ${OPENJ9_BRANCH_OPTION} ${OPENJ9_SHA_OPTION} ${OMR_REPO_OPTION} ${OMR_BRANCH_OPTION} ${OMR_SHA_OPTION}"
Copy link
Member

Choose a reason for hiding this comment

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

Should $EXTRA_GETSOURCE_OPTIONS be ${EXTRA_GETSOURCE_OPTIONS} ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sure, should be consistent.

@pshipton
Copy link
Member

pshipton commented Oct 5, 2018

This change doesn't address NOTE : For Windows platform, building openssl from source along with OpenJDK8 had a known issue and has to use pre-built openssl binary.

I expect --with-openssl=fetched --enable-openssl-bundling needs to be changed to use an existing binary and headers or at least removed from Windows until this is ready.

* [skip ci]
* Addresses eclipse-openj9#3118
* all platforms; windows 32/64, xlinux, xlinux large heap, plinux, zlinux, aix
* JDK8 only

Signed-off-by: Joe deKoning <joe_dekoning@ca.ibm.com>
@pshipton pshipton merged commit 856d92e into eclipse-openj9:master Oct 5, 2018
@pshipton
Copy link
Member

pshipton commented Oct 5, 2018

Reverting as the changes are causing build failures i.e. https://ci.eclipse.org/openj9/job/PullRequest-Compile-JDK8-linux_390-64_cmprssptrs-OpenJ9/43/console

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