Update our in-tree ICU to 60.2

RESOLVED FIXED in Firefox 59

Status

()

task
P2
normal
RESOLVED FIXED
2 years ago
3 months ago

People

(Reporter: anba, Assigned: anba)

Tracking

(Blocks 1 bug)

unspecified
mozilla59
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox58 unaffected, firefox59 fixed)

Details

Attachments

(1 attachment, 3 obsolete attachments)

Apply the patch for http://bugs.icu-project.org/trac/ticket/13462 to correct the script properties for U+3000, U+3004, U+3012, U+3020, and U+3036 in ICU60. 

The patch will be necessary to fix some test262 failures for RegExp Unicode property escapes (bug 1361876).
Flags: needinfo?(jwalden+bmo)
Priority: -- → P2
I wonder if we should also backport this commit <http://bugs.icu-project.org/trac/changeset/40670/trunk/icu4c> given that the updated code seems to be used by us (at least it looks like that when looking at our coverage data).
Assignee: nobody → andrebargull
Status: NEW → ASSIGNED
Attachment #8930044 - Flags: review?(jwalden+bmo) → review+
Flags: needinfo?(jwalden+bmo)
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/a866e058c608
Import two ICU patches from upstream. r=Waldo
Keywords: checkin-needed
Hmm, I'll guess I just drop the patch for https://ssl.icu-project.org/trac/ticket/13457 and report upstream that the patch doesn't compile for all compilers resp. with certain compiler flags...
Added a comment to [1] to notify the ICU team that the patch won't compile under GCC4.9. 

[1] https://ssl.icu-project.org/trac/ticket/13489#comment:4
Flags: needinfo?(andrebargull)
Comment on attachment 8930044 [details] [diff] [review]
bug1415827.patch

There will be an ICU maintenance release (60.2) which includes both patches. It's already tagged in svn [1], but the official release date is next week (2017-12-11) [2]. Therefore marking the current patch as obsolete and instead we'll update to 60.2.

[1] https://ssl.icu-project.org/trac/browser/tags/release-60-2/icu4c
[2] http://site.icu-project.org/download/60#TOC-ICU4C-Download
Attachment #8930044 - Attachment is obsolete: true
Update title to reflect the new goal of this bug.
Summary: Add patch for ICU ticket 13462 → Update our in-tree ICU to 60.2
Applies the current patch at http://bugs.icu-project.org/trac/ticket/13511 for AIX and uses it for all platforms to ensure the code compiles under GCC-4.9.
Attachment #8936475 - Flags: review?(jwalden+bmo)
Updates to ICU 60.2.
Attachment #8936476 - Flags: review?(jwalden+bmo)
ICU made a new 60.2 release which already includes the GCC4.9 workaround from comment #10.
Attachment #8936475 - Flags: review?(jwalden+bmo)
Attachment #8936475 - Attachment is obsolete: true
Attachment #8936476 - Attachment is obsolete: true
Attachment #8936476 - Flags: review?(jwalden+bmo)
Updates ICU to release 60.2
Attachment #8937474 - Flags: review?(jwalden+bmo)
Attachment #8937474 - Flags: review?(jwalden+bmo) → review+
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/5241bdbfa0a5
Update in-tree ICU to release 60.2. rs=Waldo
Depends on: 1429518
https://hg.mozilla.org/mozilla-central/rev/5241bdbfa0a5
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla59
Type: defect → task
You need to log in before you can comment on or make changes to this bug.