RFR(S) : 8180399 : move jdk.testlibrary.LockFreeLogManager to the top level test library

[Igor Ignatyev]

Hi Paul,

do you have other comments on this or can I consider it as reviewed?

Thanks,
-- Igor
> On May 18, 2017, at 10:36 AM, Igor Ignatyev <igor.ignatyev at oracle.com> wrote:
> 
> 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.sandoz at oracle.com <mailto:paul.sandoz at 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.ignatyev at oracle.com <mailto:igor.ignatyev at 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 <http://cr.openjdk.java.net/~iignatyev//8180397/webrev.00/index.html>
>>> JBS: https://bugs.openjdk.java.net/browse/JDK-8180397 <https://bugs.openjdk.java.net/browse/JDK-8180397>
>>> testing: affected tests
>>> 
>>> [1]  https://bugs.openjdk.java.net/browse/JDK-8075327 <https://bugs.openjdk.java.net/browse/JDK-8075327>
>>> 
>>> Thanks,
>>> -- Igor
>>> 
>>> 
>> 
>