PING: [PATCH FOR REVIEW] System Zlib Support

Xueming Shen xueming.shen at
Thu Aug 2 17:47:16 UTC 2012

Hi Andrew,

I do have a ref# for this enhancement #7110151. Though my original idea 
is to also have
a configurable mechanism to switch the zlib usage during the jvm startup 
(-D option for
example). Let's use this one for your patch. I will take a look your 
patch this week.


On 08/02/2012 03:18 AM, Andrew Hughes wrote:
> ----- Original Message -----
>> Hi all,
>> In OpenJDK 8, some support has already been added for using the
>> system installation of zlib
>> (see the thread
>> which is very similar to the support we've had in IcedTea for the
>> last five years (wow, has
>> it really been that long?).
>> This is great news for us, as it's less work we have to do in
>> upstreaming the patch (though
>> 7 still needs to be dealt with).  As is, the following webrev:
>> just fixes a few minor issues to match our existing setup, and fixes
>> a bug found when testing
>> the existing support.  In detail, the webrev:
>> * Replaces the hardcoded use of "-lz" with $(ZLIB_LIBS) and
>> $(ZLIB_CFLAGS), now set in
>> * Replaces "zlib.h" usage with<zlib.h>  (mainly to reduce difference,
>> searching '.' has no effect either way)
>> * Stops uLong being defined if SYSTEM_ZLIB is set, even if we're not
>> on Mac OS X.  Without this fix, the build fails.
>> Ok for the build forest?  If so, can I please have a bug ID for this?
>> Thanks,
>> --
>> Andrew :)
>> Free Java Software Engineer
>> Red Hat, Inc. (
>> PGP Key: 248BDC07 (
>> Fingerprint = EC5A 1F5E C0AD 1D15 8F1F  8F91 3B96 A578 248B DC07
> Any update on this?  Submission to tl, rather than build, now planned, as suggested by Alan.

More information about the core-libs-dev mailing list