Re: RFR(S): 8180195: remove jaxp testlibrary
Hi Joe, Frank, thank you for the review! Cheers, -- Igor > On May 15, 2017, at 8:21 PM, huizhe wang wrote: > > +1. Thanks Frank for checking. > > -Joe > > On 5/15/2017 7:50 PM, Frank Yuan wrote: >> Looks fine, although I am not a reviewer. >> >> Thanks >> Frank >> >>> -Original Message- >>> From: core-libs-dev [mailto:core-libs-dev-boun...@openjdk.java.net] On >>> Behalf Of Igor Ignatyev >>> Subject: RFR(S): 8180195: remove jaxp testlibrary >>> >>> http://cr.openjdk.java.net/~iignatyev//8180195/webrev.00/index.html >>>> 3049 lines changed: 114 ins; 2927 del; 8 mod; >>> Hi all, >>> >>> could you please review this small patch which removes a fork of >>> testlibrary from jaxp repo? there were a few differences b/w the >> testlibraries: >>> top level testlibrary did not have CompilerUtils class, its ProcessTools >>> did not have executeTestJava (which is basically an alias >> for >>> executeTestJvm) and its OutputAnalyzer did not have methods to dump stdout, >>> stderr into specific streams. >>> >>> this fix is a part of ongoing effort on merging and cleaning up our test >>> libraries[1]. >>> >>> bug: https://bugs.openjdk.java.net/browse/JDK-8180195 >>> webrev: http://cr.openjdk.java.net/~iignatyev//8180195/webrev.00/index.html >>> testing: :jaxp_all >>> >>> [1] https://bugs.openjdk.java.net/browse/JDK-8075327 >>> >>> Thanks, >>> -- Igor >> >
RE: RFR(S): 8180195: remove jaxp testlibrary
Looks fine, although I am not a reviewer. Thanks Frank > -Original Message- > From: core-libs-dev [mailto:core-libs-dev-boun...@openjdk.java.net] On Behalf > Of Igor Ignatyev > Subject: RFR(S): 8180195: remove jaxp testlibrary > > http://cr.openjdk.java.net/~iignatyev//8180195/webrev.00/index.html > > 3049 lines changed: 114 ins; 2927 del; 8 mod; > > Hi all, > > could you please review this small patch which removes a fork of testlibrary > from jaxp repo? there were a few differences b/w the testlibraries: > top level testlibrary did not have CompilerUtils class, its ProcessTools did > not have executeTestJava (which is basically an alias for > executeTestJvm) and its OutputAnalyzer did not have methods to dump stdout, > stderr into specific streams. > > this fix is a part of ongoing effort on merging and cleaning up our test > libraries[1]. > > bug: https://bugs.openjdk.java.net/browse/JDK-8180195 > webrev: http://cr.openjdk.java.net/~iignatyev//8180195/webrev.00/index.html > testing: :jaxp_all > > [1] https://bugs.openjdk.java.net/browse/JDK-8075327 > > Thanks, > -- Igor
Re: RFR(S): 8180195: remove jaxp testlibrary
+1. Thanks Frank for checking. -Joe On 5/15/2017 7:50 PM, Frank Yuan wrote: Looks fine, although I am not a reviewer. Thanks Frank -Original Message- From: core-libs-dev [mailto:core-libs-dev-boun...@openjdk.java.net] On Behalf Of Igor Ignatyev Subject: RFR(S): 8180195: remove jaxp testlibrary http://cr.openjdk.java.net/~iignatyev//8180195/webrev.00/index.html 3049 lines changed: 114 ins; 2927 del; 8 mod; Hi all, could you please review this small patch which removes a fork of testlibrary from jaxp repo? there were a few differences b/w the testlibraries: top level testlibrary did not have CompilerUtils class, its ProcessTools did not have executeTestJava (which is basically an alias for executeTestJvm) and its OutputAnalyzer did not have methods to dump stdout, stderr into specific streams. this fix is a part of ongoing effort on merging and cleaning up our test libraries[1]. bug: https://bugs.openjdk.java.net/browse/JDK-8180195 webrev: http://cr.openjdk.java.net/~iignatyev//8180195/webrev.00/index.html testing: :jaxp_all [1] https://bugs.openjdk.java.net/browse/JDK-8075327 Thanks, -- Igor
RFR(S): 8180195: remove jaxp testlibrary
http://cr.openjdk.java.net/~iignatyev//8180195/webrev.00/index.html > 3049 lines changed: 114 ins; 2927 del; 8 mod; Hi all, could you please review this small patch which removes a fork of testlibrary from jaxp repo? there were a few differences b/w the testlibraries: top level testlibrary did not have CompilerUtils class, its ProcessTools did not have executeTestJava (which is basically an alias for executeTestJvm) and its OutputAnalyzer did not have methods to dump stdout, stderr into specific streams. this fix is a part of ongoing effort on merging and cleaning up our test libraries[1]. bug: https://bugs.openjdk.java.net/browse/JDK-8180195 webrev: http://cr.openjdk.java.net/~iignatyev//8180195/webrev.00/index.html testing: :jaxp_all [1] https://bugs.openjdk.java.net/browse/JDK-8075327 Thanks, -- Igor