On 3/8/16 9:20 AM, Felix Yang wrote:
Joe,
thank you for the quick review.
Amy,
could you sponsor this change?
Sure, I will sponsor this for you.
Thanks,
Amy
-Felix
On 2016/3/8 2:51, joe darcy wrote:
Hello,
Looks fine; thanks,
-Joe
On 3/7/2016 8:04 AM, Felix Yang wrote:
Hi all,
please review the fix for two tests under "test/sample/".
Bug:
https://bugs.openjdk.java.net/browse/JDK-8151352
Webrev:
http://cr.openjdk.java.net/~xiaofeya/8151352/webrev.00/
Original declaration, "@library ../../../src/sample...", is invalid
with the latest change in jtreg. See
https://bugs.openjdk.java.net/browse/CODETOOLS-7901585. This fix
doesn't resolve dependency to "src/sample", but only converts them
into testng tests and declares "external.lib.roots" to avoid dot-dot.
Thanks,
Felix
