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

Michael Niedermayer michael at niedermayer.cc
Wed Jun 18 04:50:00 EEST 2025


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

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

thx

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Take away the freedom of one citizen and you will be jailed, take away
the freedom of all citizens and you will be congratulated by your peers
in Parliament.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 195 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20250618/7a5faf51/attachment.sig>


More information about the ffmpeg-devel mailing list