Hi Dannil,

On 22/05/2020 3:06 pm, Daniil Titov wrote:
Please review a webrev [1] that reverts the changes done in 
jdk.test.lib.process.OutputAnalyzer in [3].

Change [3] modified OutputAnalyzer stderrShouldBeEmptyIgnoreVMWarnings() 
methods to ignore also VM version strings . The current webrev [1] reverts this 
change

Thank you for reverting the OutputAnalyzer change.

and instead makes the tests that expects an empty stderr from launched j-* 
tools to filter out '-showversion' from test options when forwarding test VM 
option to j*-tools.

That seems workable, though I'm unclear what is adding the "-showversion" in the first place?

Thanks,
David
-----

Testing: Mach5 tier1-tier5 tests passed successfully. Tier6-tier7 tests are in 
progress.

[1]  Webrev:  http://cr.openjdk.java.net/~dtitov/8244993/webrev.01
[2] Jira issue: https://bugs.openjdk.java.net/browse/JDK-8244993
[3] https://bugs.openjdk.java.net/browse/JDK-8242009

Thank you,
Daniil


Reply via email to