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
8286623: Bundle zlib by default with JDK on macos aarch64 #8675
Conversation
👋 Welcome back jpai! A progress list of the required criteria for merging this PR into |
/label core-libs |
@jaikiran |
Dummy comment to initiate a mail to core-libs mailing list too. |
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.
Hi Jai,
Thank you for your efforts with testing and reaching out to Apple.
Given we do not see the issue with the bundled zlib, this is our best path forward for stability on macOS aarch64.
Once we can determine the cause with the help of Apple, we can switch back to the zlib that comes with macOS on this platform.
@jaikiran This change now passes all automated pre-integration checks. ℹ️ This project also has non-automated pre-integration requirements. Please see the file CONTRIBUTING.md for details. After integration, the commit message for the final commit will be:
You can use pull request commands such as /summary, /contributor and /issue to adjust it as needed. At the time when this comment was updated there had been 4 new commits pushed to the
Please see this link for an up-to-date comparison between the source branch of this pull request and the ➡️ To integrate this PR with the above commit message to the |
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.
LGTM
Thank you everyone for the reviews. |
/integrate |
Going to push as commit c3bade2.
Your commit was automatically rebased without conflicts. |
@jaikiran Do you have a plan to backport this fix to the 17 and below? |
Hello Sergey, there's currently no plan to backport this. We are hoping to hear back from Apple for the root cause issue that we have filed. |
Can I please get a review of this change to the build system which now makes bundled zlib as the default for macos aarch64 systems?
We have been running into various intermittent failures on macos aarch64 systems for several months now. After investigation we have been able to ascertain that the root cause of the issue lies within the zlib that's shipped on macos aarch64 systems. The complete details about that issue is available at https://bugs.openjdk.java.net/browse/JDK-8282954.
We have filed a bug with Apple for their inputs on what we can do to fix it in that shipped library. Given the nature of that issue, we don't have a timeline on when/if the solution for that will be available. Until that time, at least, the proposal is to use bundled zlib (which doesn't reproduce those failures) by default on macos aarch64.
Progress
Issue
Reviewers
Reviewing
Using
git
Checkout this PR locally:
$ git fetch https://git.openjdk.java.net/jdk pull/8675/head:pull/8675
$ git checkout pull/8675
Update a local copy of the PR:
$ git checkout pull/8675
$ git pull https://git.openjdk.java.net/jdk pull/8675/head
Using Skara CLI tools
Checkout this PR locally:
$ git pr checkout 8675
View PR using the GUI difftool:
$ git pr show -t 8675
Using diff file
Download this PR as a diff file:
https://git.openjdk.java.net/jdk/pull/8675.diff