[FFmpeg-devel] Bug report : build problem with "configure --disable-encoders"

Måns Rullgård mans
Tue Nov 25 00:06:06 CET 2008


Michael Niedermayer <michaelni at gmx.at> writes:

> On Mon, Nov 24, 2008 at 05:24:40PM +0100, Diego Biurrun wrote:
>> On Mon, Nov 24, 2008 at 03:08:06PM +0100, David Geldreich wrote:
>> >
>> > Le 24 nov. 08 ? 11:08, Michael Niedermayer a ?crit :
>> >
>> >> On Mon, Nov 24, 2008 at 09:56:21AM +0100, Diego Biurrun wrote:
>> >>> On Sat, Nov 22, 2008 at 03:25:47PM +0100, David Geldreich wrote:
>> >>>>
>> >>>> As it was originally in the mpegvideo_enc.c file I thought that it
>> >>>> make sense to move it to mpegvideo.c, if it is a bad idea what would
>> >>>> you propose to make --disable-encoders or --disable-decoders to  
>> >>>> work ?
>> >>>
>> >>> Michael, which file do you want?
>> >>
>> >> aandcttab.c/h
>> >
>> > Attached a patch using aandcttab.c/h to solve this --disable- 
>> > encoders/--disable-decoders issue.
>> >
>> > As I am new to ffmpeg, I have some naming/comment/doc issue :
>> >
>> > 1) I copied the copyright from another file but do not know exactly what 
>> > to put in it
>> 
>> The way you did it is right.
>> 
>
>> > 2) I do not exactly know what these aanscales table are for, so it is  
>> > hard to put a "correct" doxygen comment for the file content
>> 
>> Dunno, Michael?
>
> [...]
>
> also about the patch ...
> dependancies really are
> various en/decoders [mpeg1/2/4 ...] depend on at least one idct
> various encoders [mpeg1/2/4 ...] depend on at least one dct
> we have many dcts and idcts (and currently no way to disable any)
> if an AAN style (I)DCT is enabled then these AAN tables will be needed
> thus it is not strictly correct to add these tables to all the encoders.
> though the correct soluttion might need quite a bit of work

Such dependencies can be easily expressed in configure and the
makefiles.

-- 
M?ns Rullg?rd
mans at mansr.com




More information about the ffmpeg-devel mailing list