- Notifications
You must be signed in to change notification settings - Fork5
Commit4c5d837
committed
Fix our mapping of Windows timezones for Central America.
We were mapping "Central America Standard Time" to "CST6CDT", which seemsentirely wrong, because according to the Olson timezone database noplacein Central America observes daylight savings time on any regular basis ---and certainly not according to the USA DST rules that are implied by"CST6CDT". (Mexico is an exception, but they can be disregarded sincethey have a separate timezone name in Windows.) So, map this zone name toplain "CST6", which will provide a fixed UTC offset.As written, this patch will also result in mapping "Central AmericaDaylight Time" to CST6. I considered hacking things so that would stillmap to CST6CDT, but it seems it would confuse win32tzlist.pl to put thosetwo names in separate entries. Since there's little evidence that anysuch zone name is used in the wild, much less that CST6CDT would be a goodmatch for it, I'm not too worried about what we do with it.Per complaint from Pratik Chirania.1 parent14a1832 commit4c5d837
1 file changed
+2
-1
lines changedLines changed: 2 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
658 | 658 | | |
659 | 659 | | |
660 | 660 | | |
| 661 | + | |
661 | 662 | | |
662 | 663 | | |
663 | | - | |
| 664 | + | |
664 | 665 | | |
665 | 666 | | |
666 | 667 | | |
| |||
0 commit comments
Comments
(0)