<div dir="ltr">Pushed.</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Oct 5, 2020 at 6:26 PM Chris Johns <<a href="mailto:chrisj@rtems.org">chrisj@rtems.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 6/10/20 9:35 am, Kinsey Moore wrote:<br>
> *From:* Joel Sherrill <<a href="mailto:joel@rtems.org" target="_blank">joel@rtems.org</a>><br>
> *Sent:* Monday, October 5, 2020 17:03<br>
> *To:* Chris Johns <<a href="mailto:chrisj@rtems.org" target="_blank">chrisj@rtems.org</a>><br>
> *Cc:* Kinsey Moore <<a href="mailto:kinsey.moore@oarcorp.com" target="_blank">kinsey.moore@oarcorp.com</a>>; <a href="mailto:rtems-devel@rtems.org" target="_blank">rtems-devel@rtems.org</a><br>
> <<a href="mailto:devel@rtems.org" target="_blank">devel@rtems.org</a>><br>
> *Subject:* Re: [PATCH v1] tester: Prevent crash with json output<br>
> On Mon, Oct 5, 2020 at 4:57 PM Chris Johns <<a href="mailto:chrisj@rtems.org" target="_blank">chrisj@rtems.org</a><br>
> <mailto:<a href="mailto:chrisj@rtems.org" target="_blank">chrisj@rtems.org</a>>> wrote:<br>
> <br>
>     On 6/10/20 7:41 am, Kinsey Moore wrote:<br>
>     > This fixes an issue with json report output that causes a crash due to<br>
>     > the incorrect object member being accessed for test-too-long. This also<br>
>     > changes treatment of fatal errors to match how they are displayed in the<br>
>     > tester log output and changes "fatal error" to "fatal-error" to better<br>
>     > match the naming of all the other failure types.<br>
>     > ---<br>
>     >  tester/rt/report.py | 2 +-<br>
>     >  tester/rt/test.py   | 9 ++++++---<br>
>     >  2 files changed, 7 insertions(+), 4 deletions(-)<br>
> <br>
>     OK to push. Thanks.<br>
> <br>
> I'm OK to push this but should it also have a  ticket and go to 5?<br>
> <br>
> I'm sure that's the question Kinsey wants to hear. :)<br>
> <br>
> */[] /*<br>
> <br>
> This was necessary due to a change that didn’t go into 5, so it really shouldn’t<br>
> go into 5, either<br>
<br>
Agreed, 5 does not have the test too long support.<br>
<br>
Chris<br>
</blockquote></div>