On Wed, 12 Oct 2022 00:01:50 GMT, David Alvarez <d...@openjdk.org> wrote:

> Please, review this PR for an update of timezone data. No changes besides the 
> import were needed.

We normally run test/jdk/java/util/TimeZone/tools/share/Makefile with the JDK 
integrating new timezone data in order to generate 
test/jdk/java/util/TimeZone/TimeZoneData/{VERSION, aliases.txt, 
displaynamex.txt), which are used by some tests.  With tz2022e integrated JDK, 
the displaynames.txt seems to be updated along with the changes in tzdata2022e. 
 Since the changes are just removing the entries for Asia/Amman and 
Asia/Damascus from displaynamex.txt, the will be no impact for the test runs.  
But I would suggest these may want to be included for consistency.

diff --git a/test/jdk/java/util/TimeZone/TimeZoneData/displaynames.txt 
b/test/jdk/java/util/TimeZone/TimeZoneData/displaynames.txt
index b3823958ae4..2f2786f1c69 100644
--- a/test/jdk/java/util/TimeZone/TimeZoneData/displaynames.txt
+++ b/test/jdk/java/util/TimeZone/TimeZoneData/displaynames.txt
@@ -97,9 +97,7 @@ America/Winnipeg CST CDT
 America/Yakutat AKST AKDT
 America/Yellowknife MST MDT
 Antarctica/Macquarie AEST AEDT
-Asia/Amman EET EEST
 Asia/Beirut EET EEST
-Asia/Damascus EET EEST
 Asia/Famagusta EET EEST
 Asia/Gaza EET EEST
 Asia/Hebron EET EEST

-------------

PR: https://git.openjdk.org/jdk/pull/10666

Reply via email to