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

[Diego Biurrun]

On Sat, Nov 22, 2008 at 03:25:47PM +0100, David Geldreich wrote:
> 
> Le 21 nov. 08 ? 19:40, Diego Biurrun a ?crit :
> 
> > On Fri, Nov 21, 2008 at 01:42:18PM +0100, David Geldreich wrote:
> >>
> >> in the specific configuration "configure --disable-encoder", ffmpeg
> >> fails to build because the ff_inv_aanscales is defined in
> >> mpegvideo_enc.c and also used in eatgq.c
> >>
> >> So, I propose to move the definition of this ff_inv_aanscales table  
> >> in
> >> mpegvideo.c. Its declaration could be left in mpegvideo_enc.c and
> >> eatgq.c or moved to mpegvideo.h.
> >> I have chosen the first solution (declaration in .c files)
> >
> > Check the archives, a similar patch from Aurelien was just rejected,
> > ff_inv_aanscales is not mpeg-specific.
> 
> The table is used in the mpegvideo encoder and the eatgq decoder. For  
> --disable-encoders or --disable-decoders to work, this table should  
> not be in a coder or decoder file.
> 
> 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?

Diego