[FFmpeg-devel] [PATCH 3/3] tests/fate-run: make FATE non-interactive

Michael Niedermayer michael at niedermayer.cc
Tue Aug 11 17:35:28 CEST 2015


On Fri, Jul 31, 2015 at 08:39:00AM -0400, Ganesh Ajjanagadde wrote:
> FATE is non-interactive; it should not listen to user commands
> 
> Signed-off-by: Ganesh Ajjanagadde <gajjanagadde at gmail.com>
> ---
>  tests/fate-run.sh | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tests/fate-run.sh b/tests/fate-run.sh
> index b88730a..7aa5df2 100755
> --- a/tests/fate-run.sh
> +++ b/tests/fate-run.sh
> @@ -93,7 +93,7 @@ probeframes(){
>  
>  ffmpeg(){
>      dec_opts="-hwaccel $hwaccel -threads $threads -thread_type $thread_type"
> -    ffmpeg_args="-nostats -cpuflags $cpuflags"
> +    ffmpeg_args="-nostdin -nostats -cpuflags $cpuflags"

This is not neccessary and its not how FFmpeg is used by the majority
of users scripts AFAIK.

I dont mind if fate tests "-nostdin" but it also should test the case
without "-nostdin" as thats likely how its used most of the time

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

The real ebay dictionary, page 1
"Used only once"    - "Some unspecified defect prevented a second use"
"In good condition" - "Can be repaird by experienced expert"
"As is" - "You wouldnt want it even if you were payed for it, if you knew ..."
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150811/bfc98918/attachment.sig>


More information about the ffmpeg-devel mailing list