[Ffmpeg-devel] [patch] fix up CONFIG_ENCODERS

[Måns Rullgård]

Michael Niedermayer <michaelni at gmx.at> writes:

> Hi
>
> 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.

>> > MPlayer is compiled with --disable-mencoder all 
>> > the time and those people will benefit from skipping the code they will
>> > never need.  I agree that per-codec enabling and disabling would be cool
>> > to have but until this is becomes a reality I support updating the
>> > CONFIG_ENCODERS infrastructure.
>
> iam against wasting time on something which will be replaced
>
>>
>> Do we all agree that individual disabling of each encoder/decoder is a
>> good idea?  In that case, we might as well start the conversion, one
>> codec at a time.  Are there any codecs that depend on other codecs?
>
> various mpeg/h26* variants ...

Like I thought.  I guess the details are in the source.

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