Re: [jdk22] RFR: 8322647: Short name for the `Europe/Lisbon` time zone is incorrect

2024-01-04 Thread Joe Wang
On Thu, 4 Jan 2024 17:26:37 GMT, Naoto Sato wrote: > 8322647: Short name for the `Europe/Lisbon` time zone is incorrect Marked as reviewed by joehw (Reviewer). - PR Review: https://git.openjdk.org/jdk22/pull/30#pullrequestreview-1804685583

[jdk22] RFR: 8322647: Short name for the `Europe/Lisbon` time zone is incorrect

2024-01-04 Thread Naoto Sato
8322647: Short name for the `Europe/Lisbon` time zone is incorrect - Commit messages: - Backport ad31ec5c5f120082cedd7b9ece45b6b44147c0c5 Changes: https://git.openjdk.org/jdk22/pull/30/files Webrev: https://webrevs.openjdk.org/?repo=jdk22=30=00 Issue:

Re: RFR: 8322647: Short name for the `Europe/Lisbon` time zone is incorrect

2024-01-04 Thread Naoto Sato
On Fri, 22 Dec 2023 18:59:20 GMT, Naoto Sato wrote: > This is a regression caused by the fix to > [JDK-8317979](https://bugs.openjdk.org/browse/JDK-8317979), where the parsing > of TZDB files was incorrect. With this fix, `TimeZoneNames_en.java` which is > generated during the build time has

Re: RFR: 8322647: Short name for the `Europe/Lisbon` time zone is incorrect

2023-12-22 Thread Joe Wang
On Fri, 22 Dec 2023 21:36:32 GMT, Naoto Sato wrote: >> make/jdk/src/classes/build/tools/cldrconverter/CLDRConverter.java line 2: >> >>> 1: /* >>> 2: * Copyright (c) 2012, 2024, Oracle and/or its affiliates. All rights >>> reserved. >> >> 2024 already? ;-) > > Thanks, Joe. Not planning to

Re: RFR: 8322647: Short name for the `Europe/Lisbon` time zone is incorrect

2023-12-22 Thread Naoto Sato
On Fri, 22 Dec 2023 20:19:17 GMT, Joe Wang wrote: >> This is a regression caused by the fix to >> [JDK-8317979](https://bugs.openjdk.org/browse/JDK-8317979), where the >> parsing of TZDB files was incorrect. With this fix, `TimeZoneNames_en.java` >> which is generated during the build time

Re: RFR: 8322647: Short name for the `Europe/Lisbon` time zone is incorrect

2023-12-22 Thread Iris Clark
On Fri, 22 Dec 2023 18:59:20 GMT, Naoto Sato wrote: > This is a regression caused by the fix to > [JDK-8317979](https://bugs.openjdk.org/browse/JDK-8317979), where the parsing > of TZDB files was incorrect. With this fix, `TimeZoneNames_en.java` which is > generated during the build time has

Re: RFR: 8322647: Short name for the `Europe/Lisbon` time zone is incorrect

2023-12-22 Thread Joe Wang
On Fri, 22 Dec 2023 18:59:20 GMT, Naoto Sato wrote: > This is a regression caused by the fix to > [JDK-8317979](https://bugs.openjdk.org/browse/JDK-8317979), where the parsing > of TZDB files was incorrect. With this fix, `TimeZoneNames_en.java` which is > generated during the build time has

RFR: 8322647: Short name for the `Europe/Lisbon` time zone is incorrect

2023-12-22 Thread Naoto Sato
This is a regression caused by the fix to [JDK-8317979](https://bugs.openjdk.org/browse/JDK-8317979), where the parsing of TZDB files was incorrect. With this fix, `TimeZoneNames_en.java` which is generated during the build time has the following diffs from the previous (incorrect) one: ---