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

[andrlos]

On Fri, 11 Oct 2024 07:34:11 GMT, Christian Stein <cstein at openjdk.org> wrote:

>> @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 🤓

@sormuras can you sponsor this then please? I am not a commiter here yet :-)

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

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