[Ffmpeg-devel] [patch] fix up CONFIG_ENCODERS

[Måns Rullgård]

Roine Gustafsson <roine at users.sourceforge.net> writes:

> On Apr 17, 2005, at 4:07 PM, M?ns Rullg?rd wrote:
>> Michael Niedermayer <michaelni at gmx.at> writes:
>>> On Sunday 17 April 2005 15:11, M?ns Rullg?rd wrote:
>>>> Diego Biurrun <diego at biurrun.de> writes:
>>>>> On Fri, Apr 08, 2005 at 02:06:53PM -0600, Mike Melanson wrote:
>>>>>> Michael Niedermayer wrote:
>>>>>>> IMHO the whole CONFIG_ENCODERS should be dropped in favor of per
>>>>>>> encoder/decoder #ifdefs, the later can and should also be added
>>>>>>> incrementally, 1 codec at a time and be only around non-static
>>>>>>> functions of each codec or skip compiling whole files
>>>>>>
>>>>>>  I support this approach as well.
>>>>>
>>>>> I think a --disable-encoding (or whatever) shortcut in configure
>>>>> is and
>>>>> will be very useful.
>>>
>>> agree
>>
>> What should the rules be here?  I suppose the default should be
>> everything on.  It would then be useful to be able to
>> --disable-encoders --enable-encoder-foo and get only that single
>> encoder.  Same for decoders.
>
> You could have --enable-encoding="h264 h263 mpeg4" (implicitly
> disabling all others) and keeping --disable-encoding to disable all
> encoders. And likewise for decoders. It would also be backward
> compatible with people's current configure scripts.

That's a good idea, though I prefer comma separated names.

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