-
Notifications
You must be signed in to change notification settings - Fork 738
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
Conversation
33a2902
to
bcb6893
Compare
@@ -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' |
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.
Did you mean to remove these?
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.
No I did not. Restored.
buildenv/jenkins/common/build
Outdated
@@ -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}" |
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.
Should $EXTRA_GETSOURCE_OPTIONS
be ${EXTRA_GETSOURCE_OPTIONS}
?
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.
Sure, should be consistent.
This change doesn't address I expect |
* [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>
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 |
Signed-off-by: Joe deKoning joe_dekoning@ca.ibm.com
fyi @pshipton