Code review 8032473, Restructure JSSE regression test hierarchy in jdk test

Wang Weijun at
Mon Mar 3 05:38:06 PST 2014

Hi Xuelei

Yes it's quite difficult to read the actual webrev so instead I try to apply jdk.patch to my repo and see what happens. The patch file cannot remove the old files so I cannot be sure the cleanup is clean. Is it possible you recreate the webrev with a changeset (instead of jdk.patch). I think it can be created with a plain webrev call without using the -r option.

I just want to confirm:

1. No more file in old directory
2. JPRT runs fine
3. Number of test running not changed

Also, the webrev shows no diff and it looks each file just have a new name and its content unchanged, but we know there are at least path to store changes.


On Mar 3, 2014, at 19:46, Xuelei Fan < at> wrote:

> webrev:
> On 3/3/2014 1:54 PM, Wang Weijun wrote:
>> As Brad mentioned in the comment, do you need to update the test/TEST.groups file?
> Yes.
>> Although not friendly to read, you do have a webrev somewhere?
> Just made it ready.  Please review the update.
> Thanks,
> Xuelei
>> Thanks
>> Max

More information about the security-dev mailing list