722391 - Upgrade to zlib 1.2.6

Status: RESOLVED FIXED

(Core :: General, defect)

Description

[Daniel Holbert [:dholbert]]

Looks like zlib released a new version 1.2.6, 2 days ago.  We're currently on 1.2.5, last updated in bug 573137.

From http://www.zlib.net/ :
{{
Version 1.2.6 has many changes over 1.2.5, including these improvements:

    gzread() can now read a file that is being written concurrently
    gzgetc() is now a macro for increased speed
    Added a 'T' option to gzopen() for transparent writing (no compression)
    Added deflatePending() to return the amount of pending output
    Allow deflateSetDictionary() and inflateSetDictionary() at any time in raw mode
    deflatePrime() can now insert bits in the middle of the stream
    ./configure now creates a configure.log file with all of the results
    Added a ./configure --solo option to compile zlib with no dependency on any libraries
    Fixed a problem with large file support macros
    Fixed a bug in contrib/puff
    Many portability improvements
}}

Filing this bug to track taking the upgrade (assuming that we want to take the upgrade).

Comment 1

[Daniel Holbert [:dholbert]]

FWIW, our zlib import lives in http://mxr.mozilla.org/mozilla-central/source/modules/zlib/src/

Comment 2

[Daniel Jeter II]

Attachment: Patch to upgrade zlib to 1.2.6

Comment 3

[Daniel Jeter II]

Attachment: Updated zlib_patch.diff

Fixed a nit: Define io.h in gzguts.h when __TURBOC__ or _MSC_VER is defined.

Comment 4

[Gian-Carlo Pascutto [:gcp]]

Comment on attachment 595807 [details] [diff] [review]
Updated zlib_patch.diff

This won't move anywhere without a review.

Bug 573137 was the last update, reviewed by joe. Joe, if you're not the right one to tackle this further, can you forward to the right person?

Comment 5

[Joe Drew (not getting mail)]

Comment on attachment 595807 [details] [diff] [review]
Updated zlib_patch.diff

Review of attachment 595807 [details] [diff] [review]:
-----------------------------------------------------------------

I rubber stamp this! Time to put it on try, which I'll cause to happen.

Comment 6

[Mozilla RelEng Bot]

Autoland Patchset:
	Patches: 595807
	Branch: mozilla-central => try
	Destination: http://hg.mozilla.org/try/pushloghtml?changeset=0e450d307865
Try run started, revision 0e450d307865. To cancel or monitor the job, see: https://tbpl.mozilla.org/?tree=Try&rev=0e450d307865

Comment 7

[Mozilla RelEng Bot]

Try run for 0e450d307865 is complete.
Detailed breakdown of the results available here:
    https://tbpl.mozilla.org/?tree=Try&rev=0e450d307865
Results (out of 14 total builds):
    success: 14
Builds (or logs if builds failed) available at:
http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/[e-mail]-0e450d307865

Comment 8

[Daniel Holbert [:dholbert]]

Landed:
  https://hg.mozilla.org/integration/mozilla-inbound/rev/503a4e4804f0

Comment 9

[(no longer active)]

https://hg.mozilla.org/mozilla-central/rev/503a4e4804f0