Hi Frank,

The change looks good to me. Thanks for fixing the failure!

Best,
Joe

On 12/4/18, 6:19 PM, Frank Yuan wrote:
Hi all



Would you like to review this patch?

Bug: https://bugs.openjdk.java.net/browse/JDK-8213300

Webrev: http://cr.openjdk.java.net/~fyuan/8213300/webrev.00/



This patch made the following changes:

1.       change the path of test file from the root directory of drive C to 
C:\temp directory

2.       check if the UNC path is accessible before running the jaxp test

3.       skip the test if it's not Windows because the UNC test is only valid 
on Windows



Thanks

Frank



Reply via email to