Closed Bug 1180522 Opened 5 years ago Closed 5 years ago

Fix timezone alias for Buenos Aires

Categories

(Calendar :: Internal Components, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: MakeMyDay, Assigned: MakeMyDay)

References

Details

Attachments

(1 file)

Based on the analysis of bug 1165396, timezone alias for "Argentina Standard Time" resolves to "America/Buenos_Aires", while the zone definition only has only "America/Argentina/Buenos_Aires", which is also the correct tzid.
This patch changes the aliasTo "America/Argentina/Buenos_Aires" and adds another alias for "America/Buenos_Aires".

Furthermore, a test is added to check consistency of aliases and zone definitions. Fot this I made the alias list available from the tz service.

The test passes, see http://ftp.mozilla.org/pub/mozilla.org/thunderbird/try-builds/makemyday@gmx-topmail.de-83c7e498986e/try-comm-central-win32/try-comm-central_xp-ix_test-xpcshell-bm110-tests1-windows-build1.txt.gz
Attachment #8629703 - Flags: review?(philipp)
Comment on attachment 8629703 [details] [diff] [review]
FixTZAliasBuonesAires-V1.diff

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

Looks good to me, r=philipp

Just an idle thought I had while reviewing this, maybe we could generate zones.json at build time from the files in http://mxr.mozilla.org/comm-central/source/mozilla/intl/icu/source/data/misc/ specifically zoneinfo64.txt.

This way we could defer timezone updates to the Toolkit folks.
Attachment #8629703 - Flags: review?(philipp) → review+
Bug 1181304 filed for the automated creation of zones.json
Keywords: checkin-needed
Pushed to comm-central changeset dc4ac52c50ca
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → 4.4
You need to log in before you can comment on or make changes to this bug.