Hello Dmitry, thanks for the backport. The change looks fine to me.
Thanks, Andrew On 14/12/2020 21:12, Dmitry Cherepanov wrote: > Please review the backport for 8254177 to 7u. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8254177 > 8u commit: https://hg.openjdk.java.net/jdk8u/jdk8u/jdk/rev/93840580183a > Webrev: > http://cr.openjdk.java.net/~dcherepanov/openjdk7u/8254177/webrev.v1/ > > The patch for 8u doesn't apply cleanly to 7u. The following > modifications were done: > > 1) adjusted paths for tzdata files (make/data/tzdata -> > make/sun/javazic/tzdata) > > 2) adjusted paths for resource files > (src/share/classes/sun/util/resources/[de | es | fr | …] -> > src/share/classes/sun/util/resources) > > 3) ignored changes in files: src/aix/lib/tzmappings and > test/sun/util/calendar/* > > 4) updated the list of tzdata files in make/sun/javazic/Makefile > (excluded pacificnew/systemv) > > Sanity checked that there's no references left in the code to the > removed files pacificnew/systemv and the removed timezone US/Pacific-New. > > Ran relevant regression tests, not issues found. > > Thanks, > > Dmitry > > P.S. RFRs for updates to tzdata2020c/tzdata2020d will be submitted > separately.