Looks good! Thanks, /Staffan
> On 3 jun 2015, at 05:31, Amy Lu <amy...@oracle.com> wrote: > > lib/testlibrary/OutputAnalyzerReportingTest.java > lib/testlibrary/OutputAnalyzerTest.java > > These tests fail with jtreg4.1/b12 because jtreg4.1/b12 adds stricter > checking of @library tags and the library directory defined in the test > doesn't exist. > > Please review the fix, removed the unneeded @library. > > bug: https://bugs.openjdk.java.net/browse/JDK-8081775 > webrev: http://cr.openjdk.java.net/~amlu/8081775/webrev.00/ > > > --- old/test/lib/testlibrary/OutputAnalyzerReportingTest.java 2015-06-03 > 10:48:57.000000000 +0800 > +++ new/test/lib/testlibrary/OutputAnalyzerReportingTest.java 2015-06-03 > 10:48:57.000000000 +0800 > @@ -28,7 +28,6 @@ > * @summary Test the OutputAnalyzer reporting functionality, > * such as printing additional diagnostic info > * (exit code, stdout, stderr, command line, etc.) > - * @library /testlibrary > * @modules java.management > * @build jdk.testlibrary.* > * @run main jdk.testlibrary.OutputAnalyzerReportingTest > --- old/test/lib/testlibrary/OutputAnalyzerTest.java 2015-06-03 > 10:48:58.000000000 +0800 > +++ new/test/lib/testlibrary/OutputAnalyzerTest.java 2015-06-03 > 10:48:58.000000000 +0800 > @@ -25,7 +25,6 @@ > /* > * @test > * @summary Test the OutputAnalyzer utility class > - * @library /testlibrary > * @modules java.management > * @build jdk.testlibrary.* > * @run main jdk.testlibrary.OutputAnalyzerTest > > > Thanks, > Amy >
