Hi Naoto,
Thanks for replacing the shell script with Java code.
Given the size of the JDK, I'd suggest removing the copy at the end of
the test
unless you can rely on jtreg to remove it promptly.
The rest looks fine, Roger
On 4/5/2017 5:14 PM, Naoto Sato wrote:
I revised the test case not to rely on shell script. Please review.
http://cr.openjdk.java.net/~naoto/8177776/webrev.01/
Naoto
On 3/30/17 2:10 PM, Naoto Sato wrote:
Hello,
Please review the changes to the following issue:
https://bugs.openjdk.java.net/browse/JDK-8177776
The proposed change is located at:
http://cr.openjdk.java.net/~naoto/8177776/webrev.00/
This is for backporting fixes for JapaneseEra related issues (8054214,
8173423). The original fixes in JDK9 included a test case,
SupplementalJapaneseEraTest which is intended for the system property
testing. The above proposed fix intends to adapt that test case into
JDK8, where calendars.properties file is used instead of the system
property. The test is pretty much identical to JDK9's. The difference is
to deal with the calendars.properties file and removed some
non-applicable cases in JDK8.
Naoto
