<i18n dev> RFR: 8166875: (tz) Support tzdata2016g

[Ramanand Patil]

Hi Masayoshi,
Thank you for pointing that small miss. During testing phase, I had actually renamed "Asia/Rangoon" to "Asia/Yangon" (Asia/Rangoon entry was deleted) in TimeZoneNames*.java and this test case was failing along with other test cases, hence the bug ID tag was added there. But after correcting the TimeZoneNames*.java with correct changes I forgot to remove the bug ID from this bug.

Here is the updated Webrev: http://cr.openjdk.java.net/~rpatil/8166875/webrev.02/

Regards,
Ramanand.

-----Original Message-----
From: Masayoshi Okutsu 
Sent: Wednesday, October 05, 2016 8:49 AM
To: Ramanand Patil <ramanand.patil at oracle.com>; Martin Buchholz <martinrb at google.com>
Cc: core-libs-dev <core-libs-dev at openjdk.java.net>; i18n-dev at openjdk.java.net
Subject: Re: <i18n dev> RFR: 8166875: (tz) Support tzdata2016g

Hi Ramanand,

I don't think it's appropriate to add the bug ID to test/sun/util/resources/cldr/Bug8134384.java. This test doesn't verify the TimeZoneNames*.java changes of this fix. Otherwise, the change looks OK to me.

Thanks,
Masayoshi


On 10/4/2016 7:22 PM, Ramanand Patil wrote:
> Hi Martin,
> Thank you for your review and explanation of "Yangon". I liked the translation "End of Strife".
>
> Looking at the description of the ZoneNames.java:
> * The zid<->metazone mappings are based on CLDR metaZones.xml.
>   * The alias mappings are based on Link entries in tzdb data files.
>
> I had thought to not update this file because the CLDR metaZones.xml file doesn’t have this entry updated.
> But I think you are correct, since Link entry has this alias mentioned, there is no harm in adding these entries to zidMap and aliasMap arrays.
> Here is the updated Webrev: 
> http://cr.openjdk.java.net/~rpatil/8166875/webrev.01/
>
> Changes done:
> 	- Updated src/java.base/share/classes/java/time/format/ZoneName.java to include "Yangon" entry.
> 	- Removed unused imports from src/java.base/share/classes/java/time/format/ZoneName.java
> 	- Updated ZoneName.java in the test package as well to include "Yangon". [test/java/time/test/java/time/format/ZoneName.java]
> 	- Updated the bugID for test/java/time/test/java/time/format/TestZoneTextPrinterParser.java since this uses the "ZoneName.java" defined in test package.
>
> Also, looks like ZoneName.java is trying to maintain a comprehensive list of zone names. Though I found very few zone names are missing from this file like: "Europe/Busingen", "America/Fort_Nelson", "Antarctica/Troll" etc...
>
>
> Regards,
> Ramanand.
>
> From: Martin Buchholz [mailto:martinrb at google.com]
> Sent: Monday, October 03, 2016 8:55 PM
> To: Ramanand Patil <ramanand.patil at oracle.com>
> Cc: i18n-dev at openjdk.java.net; core-libs-dev 
> <core-libs-dev at openjdk.java.net>
> Subject: Re: <i18n dev> RFR: 8166875: (tz) Support tzdata2016g
>
> Hi Ramanand,
> Pleased to meet you!
>
> I expected to see Yangon added to ZoneName, because of the existing 
> reference to Rangoon
>
> java/time/test/java/time/format/ZoneName.java:179:        "Asia/Rangoon", "Myanmar", "Asia/Rangoon",
>
> Is ZoneName.java trying to maintain a comprehensive list of zone names?
>
> """Yangon (ရန်ကုန်) is a combination of the two words yan (ရန်) and koun (ကုန်), which mean "enemies" and "run out of", respectively. It is also translated as "End of Strife"."""
>
>
> On Mon, Oct 3, 2016 at 5:27 AM, Ramanand Patil <mailto:ramanand.patil at oracle.com> wrote:
> HI all,
> Please review the latest TZDATA integration (tzdata2016g) to JDK9.
> Bug: https://bugs.openjdk.java.net/browse/JDK-8166875
> Webrev: http://cr.openjdk.java.net/~rpatil/8166875/webrev.00/
>
> All the TimeZone related tests are passed after integration.
> [BugID is updated for tests TimeZoneTest.java and Bug8134384.java, since they verify the renamed TZID "Asia/Yangon"].
>
> Regards,
> Ramanand.