[jdk23u] RFR: 8339637: (tz) Update Timezone Data to 2024b
Naoto Sato
naoto at openjdk.org
Thu Oct 17 18:15:22 UTC 2024
On Thu, 17 Oct 2024 08:48:06 GMT, Johny Jose <duke at openjdk.org> wrote:
> Backport of tz2024b changes (https://git.openjdk.org/jdk/commit/ebc17c7c8d6febd5a887309d1b7a466bcd2cc0a9)
>
> Not clean backport as the spec changes made in mainline is not backported to jdk23u. TZdata2024 changes announced on 5th September 2024 includes changes to "EST", "MST", and "HST", i.e., changing those time zones from distinct time zones to links to other time zones. For update releases, will not make the same ZoneId changes due to current specification.
test/jdk/java/util/TimeZone/Bug6329116.java line 262:
> 260: return false;
> 261: }
> 262:
I don't think this is correct. I think COMPAT's name for "MET" needs to be modified to "CET/CEST", as MET is now a link to Europe/Brussels
test/jdk/sun/util/calendar/zi/tzdata_jdk/tz2024b_overridden_zones line 22:
> 20: # or visit www.oracle.com if you need additional information or have any
> 21: # questions.
> 22: #
A blank line would be helpful, in order to distinguish copyright from the actual comments to this file.
test/jdk/sun/util/calendar/zi/tzdata_jdk/tz2024b_overridden_zones line 23:
> 21: # questions.
> 22: #
> 23: # TZdata2024 changes announced on 5th September 2024 includes changes to
`b` is missing
test/jdk/sun/util/calendar/zi/tzdata_jdk/tz2024b_overridden_zones line 40:
> 38: Zone EST -5:00 - EST
> 39: Zone MST -7:00 - MST
> 40: Zone HST -10:00 - HST
Need a newline
-------------
PR Review Comment: https://git.openjdk.org/jdk23u/pull/191#discussion_r1805221431
PR Review Comment: https://git.openjdk.org/jdk23u/pull/191#discussion_r1805165216
PR Review Comment: https://git.openjdk.org/jdk23u/pull/191#discussion_r1805165612
PR Review Comment: https://git.openjdk.org/jdk23u/pull/191#discussion_r1805163576
More information about the jdk-updates-dev
mailing list