RFR: 7903738 : jtr.xml logs produced with -xml argument do not contain compilation failure justification

[Christian Stein]

On Tue, 8 Oct 2024 15:56:04 GMT, andrlos <duke at openjdk.org> wrote:

>> Hello @andrlos, do you have an output from the XML file which shows the difference in the output, before and after this change?
>
> @jaikiran 
> 
> before:
> 
> 
> <testcase classname="compileFail/test/CompileFail.java" name="CompileFail" time="0.438" >
>     <failure type="Failed.">
> Compilation failed: Compilation failed
> </failure>
>     </testcase>
> 
> 
> after:
> 
> <testcase classname="compileFail/test/CompileFail.java" name="CompileFail" time="0.441" >
>     <failure type="Failed.">
> Compilation failed: Compilation failed
> </failure>
>     </testcase>
>     <system-out>    </system-out>
>     <system-err>/home/jandrlik/andrlosFork/fork2/jtreg/mytests/compileFail/test/CompileFail.java:13: error: class CompileFail1 is public, should be declared in a file named CompileFail1.java
> public class CompileFail1
>        ^
> 1 error
> /home/jandrlik/andrlosFork/fork2/jtreg/mytests/compileFail/test/CompileFail.java:13: error: class CompileFail1 is public, should be declared in a file named CompileFail1.java
> public class CompileFail1
>        ^
> 1 error
> /home/jandrlik/andrlosFork/fork2/jtreg/mytests/compileFail/test/CompileFail.java:13: error: class CompileFail1 is public, should be declared in a file named CompileFail1.java
> public class CompileFail1
>        ^
> 1 error
> /home/jandrlik/andrlosFork/fork2/jtreg/mytests/compileFail/test/CompileFail.java:13: error: class CompileFail1 is public, should be declared in a file named CompileFail1.java
> public class CompileFail1
>        ^
> 1 error
> /home/jandrlik/andrlosFork/fork2/jtreg/mytests/compileFail/test/CompileFail.java:13: error: class CompileFail1 is public, should be declared in a file named CompileFail1.java
> public class CompileFail1
>        ^
> 1 error
>     </system-err>

@andrlos Once you `/integrate`d this change, I'll continue to release `jtreg` 7.5 🤓

-------------

PR Comment: https://git.openjdk.org/jtreg/pull/229#issuecomment-2406729947