[Ffmpeg-devel] dependency tracking improvements

Måns Rullgård mru
Mon Jan 22 01:21:42 CET 2007


Diego Biurrun <diego at biurrun.de> writes:

> On Mon, Jan 22, 2007 at 12:52:15AM +0100, Aurelien Jacobs wrote:
>> 
>> Here is a patch which further improve new Mans dependency tracking system.
>> It moves dependency information from configure to allcodecs.c/allformats.c.
>> Advantages are that it simplifies configure and that when you add a new
>> codec, you don't need to mess with configure. Just add a new line in
>> allcodecs.c and you're done.
>> Opinions ?
>> Can I commit this ?
>
> Putting that info in a comment sounds very fragile to me.  This way
> compilation might fail if somebody changes a comment ...

I've never liked the idea of putting essential information in comments
either.  I've seen such things go horribly wrong.

I wouldn't mind moving the dependency information to a separate file
though, if people don't like the idea of having it embedded in the
configure script.

-- 
M?ns Rullg?rd
mru at inprovide.com




More information about the ffmpeg-devel mailing list