PING: [PATCH FOR REVIEW] System Zlib Support

Alan Bateman Alan.Bateman at
Fri Aug 3 17:03:06 UTC 2012

On 03/08/2012 16:58, Andrew Hughes wrote:
> :
> It's what we've had for years, and was added orthogonally to 8 in:
> changeset:   5118:d45bc4307996
> user:        michaelm
> date:        Tue Mar 06 20:34:38 2012 +0000
> summary:     7113349: Initial changeset for Macosx port to jdk
> though that version is broken (at least on GNU/Linux) without the change
> to defines.h I posted in my patch earlier this week.
Right, that change was for Mac and it didn't have the goal to enable it 
on Linux.

BTW: Looking at your patch then I suspect it will cause problems on 
other platforms as it only sets ZLIB_LIBS in If 
folks aren't using this script to setup their environment then I'm sure 
there will be a problem on Mac at least. I don't have time to spend on 
it but I suspect Defs-macosx.gmk will need to be updated to make it the 
default as it does now.

> If IPP is binary compatible, surely setting ZLIB_LIBS/ZLIB_CFLAGS would
> be sufficient?  I don't know how different they are.
I believe it is binary compatible but I think Sherman is thinking a 
runtime knob rather than a build-time option.


More information about the core-libs-dev mailing list