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

Andreas Cadhalpun andreas.cadhalpun at googlemail.com
Thu Apr 23 20:38:15 CEST 2015


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].

Best regards,
Andreas

1: http://buildd.debian-ports.org/status/fetch.php?pkg=ffmpeg&arch=sparc64&ver=7%3A2.6.2-1&stamp=1428928967



More information about the ffmpeg-devel mailing list