<div dir="ltr">Hi Igor,<div><br></div><div>Looks good to me as well, nice job :-)</div><div>Jc</div></div><br><div class="gmail_quote"><div dir="ltr">On Fri, Sep 7, 2018 at 5:46 PM Alex Menkov <<a href="mailto:alexey.menkov@oracle.com">alexey.menkov@oracle.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Igor,<br>
<br>
Looks good.<br>
<br>
--alex<br>
<br>
On 09/07/2018 15:11, Igor Ignatyev wrote:<br>
> <a href="http://cr.openjdk.java.net/~iignatyev//8182404/webrev.00/index.html" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/~iignatyev//8182404/webrev.00/index.html</a><br>
>> 327 lines changed: 13 ins; 270 del; 44 mod;<br>
> <br>
> Hi all,<br>
> <br>
> could you please review the patch which removes jdk.testlibrary.JDKToolFinder and JDKToolLauncher and replaces their usages w/ jdk.test.lib.JDKToolFinder and JDKToolLauncher respectively?<br>
> <br>
> JBS: <a href="https://bugs.openjdk.java.net/browse/JDK-8182404" rel="noreferrer" target="_blank">https://bugs.openjdk.java.net/browse/JDK-8182404</a><br>
> webrev: <a href="http://cr.openjdk.java.net/~iignatyev//8182404/webrev.00/index.html" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/~iignatyev//8182404/webrev.00/index.html</a><br>
> testing: tier1-3 + :jdk_svc<br>
> <br>
> Thanks,<br>
> -- Igor<br>
> <br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><br></div>Thanks,<div>Jc</div></div></div>