[FFmpeg-devel] [PATCH] fate: warn if SAMPLES is not specified.

Clément Bœsch ubitux at gmail.com
Mon Dec 26 22:40:59 CET 2011


On Fri, Dec 23, 2011 at 12:33:49AM +0100, Clément Bœsch wrote:
[...]
> > Hm, I don't think that should be necessary to change, just specifying
> > the additional fate-samples-warning dependency should work.
> > However particularly with -j <n> it can be printed whenever.
> > (I am not completely sure about the ':' vs. '::' syntax, might be you
> > have to switch to using ::, but at least duplicating it completely
> > should not be necessary).
> > Alternatively you could just do a
> > FATE += fate-samples-warning
> > but that would probably print it at the end - whether that is better or
> > worse would be up to discussion.
> 
> See new attached patch. The "fate: $(FATE)" line needed to be changed
> using '::', or it wouldn't work.
> 
> -- 
> Clément B.

> From 72fad0fa6daedf6dcafc89e1272aad0cac819bd2 Mon Sep 17 00:00:00 2001
> From: =?UTF-8?q?Cl=C3=A9ment=20B=C5=93sch?= <ubitux at gmail.com>
> Date: Thu, 22 Dec 2011 16:22:48 +0100
> Subject: [PATCH] fate: warn if SAMPLES is not specified.
> 
> ---
>  tests/Makefile |    4 +++-
>  1 files changed, 3 insertions(+), 1 deletions(-)
> 

Pushed this version; flame me on cvslog if any problem.

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20111226/e1c0c326/attachment.asc>


More information about the ffmpeg-devel mailing list