[9] request for review 8072578: ProbeKeystores.java creates files in test.src

Weijun Wang weijun.wang at oracle.com
Wed May 20 09:59:41 UTC 2015


The code change looks fine.

If it were me, I won't do any clean up.

--Max

On 5/14/2015 11:14 PM, Vincent Ryan wrote:
> That makes more sense. I’ve updated the webrev in place.
>
>
>> On 14 May 2015, at 15:29, Weijun Wang <weijun.wang at oracle.com> wrote:
>>
>> Hi Vinnie
>>
>> Is there any reason why they cannot be simply created in the current directory?
>>
>> In fact, if you put them there, then you don't need to clean them up because jtreg will do that for you. If the test fails some day, they will be retained for your diagnostics.
>>
>> Thanks
>> Max
>>
>> On 5/13/2015 11:52 PM, Vincent Ryan wrote:
>>> Please review this test correction to avoid creating temporary keystore
>>> files in the test source directory.
>>> Thanks.
>>>
>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8072578
>>> Webrev: http://cr.openjdk.java.net/~vinnie/8072578/webrev.00/
>>>
>>>
>


More information about the security-dev mailing list