RFR(XXS) : 8210108 : sun/tools/jstatd test build failures after JDK-8210022

Mikael Vidstedt mikael.vidstedt at oracle.com
Tue Aug 28 20:19:55 UTC 2018


Looks good, thanks for fixing!

Cheers,
Mikael

> On Aug 28, 2018, at 11:57 AM, Igor Ignatyev <igor.ignatyev at oracle.com> wrote:
> 
> http://cr.openjdk.java.net/~iignatyev//8210108/webrev.00/index.html
>> 3 lines changed: 1 ins; 1 del; 1 mod; 
> 
> 
> Hi all,
> 
> could you please review this small and trivial follow-up fix?
> 
> unfortunately, test/jdk/sun/tools/jstatd/JstatdTest now uses two different OutputAnalyzer classes, jdk.test.lib.process.OutputAnalyzer which it gets from jdk.test.lib.process.ProcessThread, and jdk.testlibrary.OutputAnalyzer from jdk.testlibrary.ProcessTools. as JstatdTest uses j.t.OutputAnalyzer more often, the fix restores import statement and uses FQDN where j.t.l.process.OutputAnalyzer is used.
> 
> // we will get rid of this FQDN, as soon as jdk.testlibrary.ProcessTools is "merged" w/ jdk.t.l.process.ProcessTools, which is tracked by 8210112.
> 
> JBS: https://bugs.openjdk.java.net/browse/JDK-8210108
> webrev: http://cr.openjdk.java.net/~iignatyev//8210108/webrev.00/index.html
> testing: sun/tools/jstatd tests
> 
> Thanks,
> -- Igor



More information about the serviceability-dev mailing list