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 building openssl on AIX #3189

Merged
merged 1 commit into from
Oct 9, 2018
Merged

Conversation

pshipton
Copy link
Member

@pshipton pshipton commented Oct 9, 2018

Reverts part of #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

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>
@keithc-ca keithc-ca self-assigned this Oct 9, 2018
@keithc-ca keithc-ca self-requested a review October 9, 2018 15:33
@keithc-ca
Copy link
Contributor

Jenkins test sanity aix jdk8

@pshipton
Copy link
Member Author

pshipton commented Oct 9, 2018

note that PR testing will not take the changes to default.yml into account (or so I was told)

@keithc-ca
Copy link
Contributor

I see this in the build log

00:29:02.887 + bash configure --with-freemarker-jar=/home/jenkins/freemarker.jar --with-boot-jdk=/usr/java7 --with-cups-include=/opt/freeware/include --disable-ccache --with-jobs=8 --with-openssl=fetched --enable-openssl-bundling

which suggests it is taking effect.

@pshipton
Copy link
Member Author

pshipton commented Oct 9, 2018

thanks, clearly my information was incorrect, which will make it easy to test future changes.

@keithc-ca
Copy link
Contributor

Not waiting for the travis build which should not be affected by this.

@keithc-ca keithc-ca merged commit 150b8af into eclipse-openj9:master Oct 9, 2018
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