RFR: JDK-8067945: SVC jdk/test/* should be cleaned from JRE layout dependency (corrected per the review findings)

[Alexander Kulyakhtin]

Hi Katia,

Please, find attached the jdk.patch containing the changes per your findings. The patch has been made by running the webrev tool.

Best regards,
Alex 

----- Original Message -----
From: yekaterina.kantserova at oracle.com
To: alexander.kulyakhtin at oracle.com, serviceability-dev at openjdk.java.net
Sent: Wednesday, January 21, 2015 12:11:58 PM GMT +04:00 Abu Dhabi / Muscat
Subject: Re: RFR: JDK-8067945: SVC jdk/test/* should be cleaned from JRE layout dependency (corrected per the review findings)

Hi Alex,

Comments bellow should be removed. You don't need to make a webrev for 
it, only the changes are included in the final patch.

test/com/sun/jdi/connect/spi/JdiLoadedByCustomLoader.java

39         // create files from given arguments and tools.jar

test/com/sun/tools/attach/BasicTests.java

81         // Need to add jdk/lib/tools.jar to classpath.


Best regards,
Katja (not a reviewer)



On 01/19/2015 05:00 PM, Alexander Kulyakhtin wrote:
> Hi,
>
> Could you, please, review the fix below.
>
> To adress the previous review findings any referenes to "test.jdk" have been removed.
>
> bug: https://bugs.openjdk.java.net/browse/JDK-8067945
> webrev: http://cr.openjdk.java.net/~eistepan/~akulyakhtin/8067945/webrev.01/
>
> References to tools.jar are removed from the tests as jdk9 drops tools.jar
>
> Thanks,
> Alex

-------------- next part --------------
A non-text attachment was scrubbed...
Name: jdk.patch
Type: text/x-patch
Size: 8600 bytes
Desc: not available
URL: <http://mail.openjdk.java.net/pipermail/serviceability-dev/attachments/20150121/1e68a080/jdk.patch>