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

Additional 64 bit support request for openJDK #426

Open
LanceAndersen opened this issue Jun 27, 2019 · 5 comments
Open

Additional 64 bit support request for openJDK #426

LanceAndersen opened this issue Jun 27, 2019 · 5 comments

Comments

@LanceAndersen
Copy link

LanceAndersen commented Jun 27, 2019

If possible, could you please consider adapting the changes below into the next release of zlib as we need to make these changes when we take the latest zlib release. Once these changes are incorporated, we should be able to take releases un-modified.

Thank you for your consideration.

Zlib-OpenJDK-Changes.txt

@madler
Copy link
Owner

madler commented Jun 27, 2019

Why do you need to make these changes?

@LanceAndersen
Copy link
Author

I was not involved in the original build changes to OpenJDK/Java SE, but this was needed to support some of the platforms openJDK runs on as described in http://gcc.gnu.org/onlinedocs/cpp/Common-Predefined-Macros.html#Common-Predefined-Macros:

_LP64

These macros are defined, with value 1, if (and only if) the compilation is for a target where long int and pointer both use 64-bits and int uses 32-bit.

@jiaz83
Copy link

jiaz83 commented Apr 11, 2022

We've recently had many users report issues that might be connected to this.
While many linux distro just backported the latest security fix to 1.2.11, it looks like arch linux (and others?) just updated to 1.2.12 and now java fails when using CRC32 or opening jar files,see
bbs.archlinux.org/viewtopic.php?id=275421
stackoverflow.com/questions/71757009/strange-java-behaviour-with-zip-files-including-jars
As openjdk builds rely on system installed zlib, I think this is fairly important change to fix and release as breaks java usage on those systems

@jiaz83
Copy link

jiaz83 commented Apr 11, 2022

I've just got feedback from arch linux user that downgrading back to 1.2.11 fixes the issue with java

@katnatek
Copy link

Thomas Backlund from mageia fix this issue backporting ec3df00 to 1.2.12

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

No branches or pull requests

4 participants