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

[Yekaterina Kantserova]

Looks god to me (not a reviewer).

// Katja

On 01/21/2015 11:23 AM, Alexander Kulyakhtin wrote:
> 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