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

Use --with-openssl=fetched on Windows #14343

Merged
merged 2 commits into from
Jan 24, 2022
Merged

Conversation

keithc-ca
Copy link
Contributor

@keithc-ca keithc-ca commented Jan 21, 2022

Issue: #14302:

  • update build variables
  • update build instructions

@keithc-ca
Copy link
Contributor Author

jenkins compile win,win32 jdk8 depends ibmruntimes/openj9-openjdk-jdk8#543

@keithc-ca
Copy link
Contributor Author

jenkins compile win32 jdk8 depends ibmruntimes/openj9-openjdk-jdk8#543

* variables-functions.groovy: look for '--with-openssl' option
  in EXTRA_CONFIGURE_OPTIONS; don't download if absent, 'fetched'
  or 'system'

* defaults.yml: update configure options

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

jenkins compile win,win32 jdk8 depends ibmruntimes/openj9-openjdk-jdk8#543

@keithc-ca
Copy link
Contributor Author

jenkins compile win jdk11 depends ibmruntimes/openj9-openjdk-jdk11#471

@keithc-ca
Copy link
Contributor Author

jenkins compile win jdk17 depends ibmruntimes/openj9-openjdk-jdk17#61

@keithc-ca
Copy link
Contributor Author

jenkins compile win jdk18 depends ibmruntimes/openj9-openjdk-jdk18#9

@keithc-ca
Copy link
Contributor Author

jenkins compile win jdknext depends ibmruntimes/openj9-openjdk-jdk#395

@keithc-ca keithc-ca marked this pull request as ready for review January 22, 2022 03:42
@pshipton
Copy link
Member

jenkins compile all jdk8 depends ibmruntimes/openj9-openjdk-jdk8#543

@pshipton
Copy link
Member

jenkins compile all jdk11 depends ibmruntimes/openj9-openjdk-jdk11#471

@pshipton
Copy link
Member

jenkins compile all jdk17 depends ibmruntimes/openj9-openjdk-jdk17#61

@pshipton
Copy link
Member

There are problems on other platforms.

@keithc-ca
Copy link
Contributor Author

jenkins compile all jdk8 depends ibmruntimes/openj9-openjdk-jdk8#543

@pshipton
Copy link
Member

jenkins compile all jdk11 depends ibmruntimes/openj9-openjdk-jdk11#471

@pshipton
Copy link
Member

jenkins compile all jdk17 depends ibmruntimes/openj9-openjdk-jdk17#61

@keithc-ca
Copy link
Contributor Author

AIX seems to have some strange (unrelated?) problems:

13:46:37  === Output from failing command(s) repeated here ===
13:46:37  * For target buildtools_depend__the.COMPILE_DEPEND_batch:
13:46:37  Sjavac server failed to initialize: Deadlock condition if locked
13:46:37  Process output:
13:46:37  <End of process output>
13:46:37  IOException caught during compilation: Server failed to initialize: Deadlock condition if locked
13:46:37  
13:46:37  * All command lines available in /home/jenkins/workspace/Build_JDK11_ppc64_aix_Personal/build/aix-ppc64-normal-server-release/make-support/failure-logs.
13:46:37  === End of repeated output ===

Both openssl and openj9 appear to have compiled successfully.

@keithc-ca
Copy link
Contributor Author

jenkins compile aix jdk11 depends ibmruntimes/openj9-openjdk-jdk11#471

@pshipton
Copy link
Member

@jdekonin the AIX build that failed happened on p8-java1-ibm03. Maybe these machines aren't as ready to go as we hoped. Or maybe they are just faster and expose a timing hole. The failure isn't new, we see it occasionally, but not very often.
https://openj9-jenkins.osuosl.org/job/Build_JDK11_ppc64_aix_Personal/216/

@pshipton
Copy link
Member

The new build is running on the same machine, we'll see if it works.
https://openj9-jenkins.osuosl.org/job/PullRequest-OpenJ9/1466/

Either way, it's not an issue with this PR.

@pshipton pshipton merged commit f07b484 into eclipse-openj9:master Jan 24, 2022
@keithc-ca keithc-ca deleted the openssl branch January 24, 2022 20:22
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.

None yet

2 participants