> On 17 Oct 2016, at 09:51, Srinivasan Raghavan 
> <srinivasan.ragha...@oracle.com> wrote:
> 
> Hi all
> 
> Please review the fix for the bug 
> 
> Bug :https://bugs.openjdk.java.net/browse/JDK-8075205
> 
> The tests uses classes directory for the output files. This can result in the 
> files being left over after the test is complete which can result in 
> instability. The tests copies the files to be compiled form test src to test 
> classes which can result in copy of permission and result in instability 
> because the test has delete operations. The test fails randomly mostly in 
> copy or delete operation. I propose the test to be refactored to make the use 
> scratch directory as its output directory and eliminate shell by using 
> testlibrary utils.
> 
> fix : http://cr.openjdk.java.net/~sraghavan/8075205/webrev.00/

This looks good to me. Thanks Srinivasan.

-Chris.

Reply via email to