[FFmpeg-devel] [PATCH] lavd: add uncoded_frame test program.

Clément Bœsch u at pkh.me
Sat Feb 8 18:09:27 CET 2014


On Sat, Feb 08, 2014 at 05:31:24PM +0100, Nicolas George wrote:
> Le decadi 20 pluviôse, an CCXXII, Clément Bœsch a écrit :
> > Why is this part of the PROGS? PROGS are for the main fftools. Shouldn't
> > it be TESTPROGS?
> 
> This was discussed earlier: the build system for TESTPROGS does not support
> inter-library dependencies, and therefore can only handle test programs that
> work on a very small part of the library.
> 

Sorry I miss the discussion...

> If you have a better idea to make this work, do not hesitate.
> 
> > Also, why not make it part of the examples instead?
> 
> The one of the purpose is to be able to build a FATE test on top of this. I
> do not think the examples can fit that purpose.
> 

What would prevent a FATE test to depend on CONFIG_UNCODED_FRAME_EXAMPLE?

Having the tool in PROGS doesn't look like a wise solution to me... I'm
not comfortable having this tool half mixed with the fftools build system
at all.

-- 
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/20140208/02be9eee/attachment.asc>


More information about the ffmpeg-devel mailing list