[FFmpeg-devel] [FATE PATCH] Properly list test errors when there are no failed tests

[Kacper Michajlow]

On Fri, Jun 27, 2025, 14:44 Martin Storsjö <martin at martin.st> wrote:

> On Wed, 18 Jun 2025, Michael Niedermayer wrote:
>
> > Hi Martin
> >
> > On Fri, Jun 13, 2025 at 01:57:05PM +0300, Martin Storsjö wrote:
> >> If there were failures while running tests, e.g. if failing to
> >> compile checkasm or any other of the test programs, there are no
> >> failed tests per se, and the number of succeessful tests is
> >> equal to the total number of tests.
> >>
> >> For these cases, check the job status code instead of declaring
> >> them as a full success.
> >> ---
> >>  history.cgi | 2 +-
> >>  index.cgi   | 2 +-
> >>  report.cgi  | 6 +++++-
> >>  3 files changed, 7 insertions(+), 3 deletions(-)
> >
> > sounds reasonable, but ill leave the perl review to someone who actually
> > knows perl
>
> I've tested it locally, so if we have nobody else volunteering to review
> it, is that enough?
>
> > Its a few years since the last change to fateserver, i think we dont
> > have a script to update the checkout on the server from a git push.
> > Its a while, i dont remember exactly :)
> > so you have to update that checkout after pushing possibly
>
> I don't think I have access to any such server, but if it's ok with you
> can I (try to) push this patch to the gitosis repo, and then ask some
> admin (Timo?) to possibly update a checkout somewhere.
>
> // Martin
>

There is also https://fatebeta.ffmpeg.org/, should it be updated for parity
with perl version?

- Kacper

>