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 the system zlib on platforms other than Windows #692

Merged
merged 1 commit into from
Sep 13, 2023

Conversation

pshipton
Copy link
Member

@pshipton pshipton commented Sep 12, 2023

This makes jdk8 consistent with jdk11.

Closes eclipse-openj9/openj9#18127

@pshipton
Copy link
Member Author

jenkins compile aix,zlinux jdk8

@pshipton
Copy link
Member Author

AIX checking for which zlib to use... system
zlinux checking for which zlib to use... system

@pshipton pshipton requested a review from keithc-ca September 12, 2023 19:19
@pshipton
Copy link
Member Author

The PR build is https://openj9-jenkins.osuosl.org/job/PullRequest-OpenJDK8/93/
Pushing a change to the commit wording.

@pshipton pshipton changed the title Use the system zlib on Linux platform Use the system zlib on Linux platforms Sep 12, 2023
This makes jdk8 consistent with jdk11.

Issue eclipse-openj9/openj9#18127

Signed-off-by: Peter Shipton <[email protected]>
@pshipton pshipton changed the title Use the system zlib on Linux platforms Use the system zlib on platforms other than Windows Sep 13, 2023
@keithc-ca
Copy link
Member

Jenkins compile xlinux,win32 jdk8

@keithc-ca
Copy link
Member

In https://openj9-jenkins.osuosl.org/job/Build_JDK8_x86-32_windows_Personal/737/consoleText:

[2023-09-13T14:43:21.084Z] checking for which zlib to use... bundled

In https://openj9-jenkins.osuosl.org/job/Build_JDK8_x86-64_linux_Personal/462/consoleText:

[2023-09-13T14:42:44.156Z] checking for which zlib to use... system

@keithc-ca keithc-ca merged commit 2fab540 into ibmruntimes:openj9 Sep 13, 2023
keithc-ca added a commit to keithc-ca/openj9-openjdk8 that referenced this pull request Sep 18, 2023
With the recent preference for --with-zlib=system (see ibmruntimes#692), we need
to back-port

* 8140223: fix the build with a toolchain with a linker defaulting
  to ld --as-needed

I'm not sure why builds succeed on RHEL, but they fail on Ubuntu.

Signed-off-by: Keith W. Campbell <[email protected]>
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.

zlib bundling
2 participants