(10) RFR of JDK-8181912,Refactor locale related shell test test/java/io/File/MacPathTest.sh to java test

Hamlin Li huaming.li at oracle.com
Thu Jun 15 03:35:04 UTC 2017


On 2017/6/15 1:22, Alan Bateman wrote:
> On 12/06/2017 09:00, Hamlin Li wrote:
>> Would you please review the below patch?
>>
>> bug: https://bugs.openjdk.java.net/browse/JDK-8181912
>>
>> webrev: http://cr.openjdk.java.net/~mli/8181912/webrev.00/
>>
> Are you sure that setting the user.* properties on the command 
> actually works? I assume we'll have to change this back at some point 
> to launch the test with LC_ALL set to UTF-8.
Hi Alan,

Besides of setting user.language and user.country, the patch also sets 
file.encoding, and sun.jnu.encoding as UTF-8.

I understand your concern: we're not sure if setting -Dxxx properties 
will have totally same functionality as setting env variable LC_ALL. I 
have no answer for this question.
What tests have been done:
  1. set user.* properties will affect what Locale.getDefault() gets, 
set file.encoding will affect what 
java.nio.charset.Charset.defaultCharset() gets.
  2. jprt passed.
  3. test on some other mac machine in SQE pool.

Do you suggest we should keep this test as shell?

Thank you
-Hamlin
>
> -Alan



More information about the core-libs-dev mailing list