Closed Bug 1513934 Opened 4 years ago Closed 3 years ago
Import patch for ICU-20302
https://unicode-org.atlassian.net/browse/ICU-20302 (<https://github.com/unicode-org/icu/pull/315>) fixes a time zone detection issue on Windows 7. We should import the patch in our local ICU copy. Hat tip to Jeff Genovy for pinging us in the ICU bug report!
This issue only affects ICU 63, so I guess we want to backport the patch to Beta, given that Beta uses ICU 63 (bug 1499026).
Imported the two PRs via ``` wget https://github.com/unicode-org/icu/pull/315.patch -O bug-1513934-timezone-detection-win7-part1.diff wget https://github.com/unicode-org/icu/pull/318.patch -O bug-1513934-timezone-detection-win7-part2.diff ``` and then reimported the ICU-63 maintenance branch. That means we also get <https://github.com/unicode-org/icu/pull/283> to fix a performance regression in UnicodeSet <https://unicode-org.atlassian.net/browse/ICU-20250>.
Attachment #9031432 - Flags: review?(jwalden+bmo)
Waldo, can you please prioritize this review as it's a patch targeting Fx65 and we're already halfway through the Beta cycle?
Comment on attachment 9031432 [details] [diff] [review] bug1513934.patch [Beta/Release Uplift Approval Request] Feature/Bug causing the regression: None User impact if declined: Potential problems with time zone detection on Windows 7 -- exactly what problems, seems to depend on the state of the program and other things. Is this code covered by automated tests?: No Has the fix been verified in Nightly?: No Needs manual test from QE?: No If yes, steps to reproduce: List of other uplifts needed: None Risk to taking this patch: Medium Why is the change risky/not risky? (and alternatives if risky): This is importing fixes from upstream, so that probably mitigates the risk a bit. But this does touch a bit of code (mostly for the fixes that are *not* time zone-related, that were already on the maintenance branch), so it's not the targeted patch it could be. String changes made/needed: N/A
Attachment #9031432 - Flags: approval-mozilla-beta?
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/mozilla-inbound/rev/6ac79113d538 Import ICU patch to fix possible time zone misdetection on Windows 7. r=Waldo
Comment on attachment 9031432 [details] [diff] [review] bug1513934.patch [Triage Comment] Fixes problems with time zone detection on Windows 7 caused by the most recent ICU update. Approved for 65.0b8.
Attachment #9031432 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
You need to log in before you can comment on or make changes to this bug.