[FFmpeg-devel] [PATCH 1/2] tests/tiny_psnr: do not ignore errors from run_psnr

[Michael Niedermayer]

On Thu, Apr 23, 2015 at 08:38:15PM +0200, Andreas Cadhalpun wrote:
> On 23.04.2015 15:44, Michael Niedermayer wrote:
> > On Wed, Apr 22, 2015 at 09:10:51PM -0700, Timothy Gu wrote:
> >> On Thu, Apr 23, 2015 at 04:43:09AM +0200, Michael Niedermayer wrote:
> >>> failure to calculate psnr should not result in tiny_psnr returning success
> >>>
> >>> Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
> >>> ---
> >>>  tests/tiny_psnr.c |    9 ++++++---
> >>>  1 file changed, 6 insertions(+), 3 deletions(-)
> >>
> >> Patchset LGTM. Would be cool if you can add a link to a broken FATE results
> >> page in the second patch commit message though.
> > 
> > i can add a link if someone has one, i seem not lucky in finding one
> > quickly ATM
> 
> Does this fix the "awk: line 1: syntax error" messages?
> Those can be seen among the test failures on sparc64 [1].

probably but it wont fix the fate failures just the nonsense errors

applied

Thanks to all

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

No great genius has ever existed without some touch of madness. -- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150424/179a5d4e/attachment.asc>