Paul, doh, too many review requests in parallel, thank you for providing the correct links.
yes, you are right j/l/m/ThreadMXBean/Locks test does not depend on jdk testlibrary anymore, I have cleaned it up and retested: > * @author Mandy Chung > * @author Jaroslav Bachorik > * > - * @library /lib/testlibrary /test/lib > + * @library /test/lib > * > - * @build jdk.testlibrary.* > * @run main/othervm Locks > */ > import java.lang.management.*; Thanks, -- Igor > On May 17, 2017, at 5:13 PM, Paul Sandoz <paul.san...@oracle.com> wrote: > > > It’s this one :-) > > https://bugs.openjdk.java.net/browse/JDK-8180399 > <https://bugs.openjdk.java.net/browse/JDK-8180399> > http://cr.openjdk.java.net/~iignatyev/8180399/webrev.00/ > <http://cr.openjdk.java.net/~iignatyev/8180399/webrev.00/> > > Does the Locks test need to depend on stuff in /lib/testlibrary ? > > Paul. > >> On 16 May 2017, at 21:21, Igor Ignatyev <igor.ignat...@oracle.com >> <mailto:igor.ignat...@oracle.com>> wrote: >> >> http://cr.openjdk.java.net/~iignatyev//8180399/webrev.00/index.html >> <http://cr.openjdk.java.net/~iignatyev//8180399/webrev.00/index.html> >>> 186 lines changed: 87 ins; 90 del; 9 mod; >> >> Hi all, >> >> could you please review this small patch which moves LockFreeLogManager >> class from jdk test library to the top level test library? >> this fix is a part of ongoing effort on merging and cleaning up our test >> libraries[1]. >> >> webrev: http://cr.openjdk.java.net/~iignatyev//8180397/webrev.00/index.html >> JBS: https://bugs.openjdk.java.net/browse/JDK-8180397 >> testing: affected tests >> >> [1] https://bugs.openjdk.java.net/browse/JDK-8075327 >> >> Thanks, >> -- Igor >> >> >
