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

[Michael Niedermayer]

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:
> > 
> > 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?

aandcttab.c/h

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

In a rich man's house there is no place to spit but his face.
-- Diogenes of Sinope
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20081124/28c93876/attachment.pgp>