[Ffmpeg-devel] dependency tracking improvements
Mon Jan 22 10:08:00 CET 2007
Diego Biurrun <diego at biurrun.de> writes:
> On Mon, Jan 22, 2007 at 01:21:21AM +0100, Aurelien Jacobs wrote:
>> On Mon, 22 Jan 2007 01:06:18 +0100
>> Diego Biurrun <diego at biurrun.de> wrote:
>> > 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 ...
>> Anyway, we already heavily rely on the structure of this file to parse
>> it with sed. It's already very easy to break.
>> But if you think using comments is bad, maybe we could do something
>> like this:
>> - REGISTER_DECODER(AAC, aac);
>> + REGISTER_DECODER(AAC, aac, libfaad);
>> Do you prefer it ?
>> IMO, it's as much fragile, and uglier than a comment.
> That's better and less fragile.
But it doesn't allow the current dependencies to be correctly expressed.
mru at inprovide.com
More information about the ffmpeg-devel