Closed Bug 1328099 Opened 8 years ago Closed 8 years ago

Update in-tree zlib to version 1.2.11

Categories

(Core :: General, defect)

Product:
Core
Component:
General
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla53
Tracking Flags:
Tracking Status
firefox53 --- fixed

People

(Reporter: glennrp+bmo, Assigned: glennrp+bmo)

References

(
URL
)

Details

Attachments

(1 file, 6 obsolete files)

v02-1328099-zlib-1.2.11
273.59 KB, patch
jrmuizel
: review+
Details | Diff | Splinter Review
zlib-1.2.9 was released yesterday, 31 December 2016. We should update the embedded zlib.
Assignee: nobody → glennrp+bmo
URL: zlib.net
Status: NEW → ASSIGNED
See Also: → 1326234
zlib-1.2.9 was found to be buggy and zlib-1.2.10 has been released.
Summary: Update in-tree zlib to version 1.2.9 → Update in-tree zlib to version 1.2.10
Building with zlib-1.2.9 exposed a new bug in libpng-1.6.27 which has been fixed. Marking dependence upon libpng-1.6.28
Depends on: 1328354
zlib-1.2.11 has been released to fix a bug in 1.2.10.
Summary: Update in-tree zlib to version 1.2.10 → Update in-tree zlib to version 1.2.11
Attached patch v00-1328099-zlib-1.2.11 (obsolete) — — Splinter Review
Attached patch v01-1328099-zlib-1.2.11 (obsolete) — — Splinter Review
Added macros to mozzconf.h for new exported symbols in zlib-1.2.11
Attachment #8827305 - Attachment is obsolete: true
Please "try" the v01 patch, after libpng-1.6.28 lands.
Flags: needinfo?(ryanvm)
Parsing...Patch id=8827305 desc="v00-1328099-zlib-1.2.11" diff data were discarded: UnicodeDecodeError: 'utf8' codec can't decode byte 0xa7 in position 5774: invalid start byte Patch id=8827311 desc="v01-1328099-zlib-1.2.11" diff data were discarded: UnicodeDecodeError: 'utf8' codec can't decode byte 0xa7 in position 5774: invalid start byte
Flags: needinfo?(ryanvm) → needinfo?(glennrp+bmo)
Removed non-ASCII symbols from ChangeLog Please "try" the v02 patch
Attachment #8827311 - Attachment is obsolete: true
Flags: needinfo?(glennrp+bmo) → needinfo?(ryanvm)
I'm still hitting failures trying to apply it :( Parsing...Patch id=8827305 desc="v00-1328099-zlib-1.2.11" diff data were discarded: UnicodeDecodeError: 'utf8' codec can't decode byte 0xa7 in position 5774: invalid start byte Patch id=8827311 desc="v01-1328099-zlib-1.2.11" diff data were discarded: UnicodeDecodeError: 'utf8' codec can't decode byte 0xa7 in position 5774: invalid start byte Patch id=8827510 desc="v02-1328099-zlib-1.2.11" diff data were discarded: UnicodeDecodeError: 'utf8' codec can't decode byte 0xa7 in position 5774: invalid start byte
Flags: needinfo?(ryanvm) → needinfo?(glennrp+bmo)
Attachment #8827510 - Attachment is obsolete: true
Flags: needinfo?(glennrp+bmo)
Please "try" all 4 parts to try to isolate the charset problem
Flags: needinfo?(ryanvm)
All 4 apply cleanly, actually. And when I saved v02 of the full patch locally and then tried to import it, that also worked OK. I suspect qimportbz was just getting confused by the older attachments on the bug. Here's a Try push for the full v02 patch, sorry for the confusion: https://treeherder.mozilla.org/#/jobs?repo=try&revision=052dfabc5677fb57db46a839acad7a90fafc8734
Flags: needinfo?(ryanvm)
Attachment #8827510 - Attachment is obsolete: false
Attachment #8827547 - Attachment is obsolete: true
Attachment #8827549 - Attachment is obsolete: true
Attachment #8827552 - Attachment is obsolete: true
Comment on attachment 8827554 [details] [diff] [review] v03-1328099-part04-zlib-1.2.11.diff Marking v03 parts 1-4 obsolete and restoring v02
Attachment #8827554 - Attachment is obsolete: true
Comment on attachment 8827510 [details] [diff] [review] v02-1328099-zlib-1.2.11 Try looks green enough.
Attachment #8827510 - Flags: review?(jmuizelaar)
Attachment #8827510 - Flags: review?(jmuizelaar) → review+
Pushed by ryanvm@gmail.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/749b7caa7b62 Update in-tree zlib to version 1.2.11. r=jrmuizel
https://hg.mozilla.org/mozilla-central/rev/749b7caa7b62
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox53: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: