[FFmpeg-devel] [PATCH] h263.c needs mpegvideo_enc.c

[Benoit Fouet]

Carl Eugen Hoyos wrote:
> Hi!
>
> Aurelien Jacobs <aurel <at> gnuage.org> writes:
>
>   
>>>> h263.c contains a call to ff_write_quant_matrix which is defined in
>>>> mpegvideo_enc.c, so I think the attached patch to compile
>>>> mpegvideo_enc.c unconditionally makes sense.
>>>>         
>>> Even more, if the conditional compilation is removed.
>>>       
>> No, please don't.
>> I will solve this properly when I will finalize making h263.o and
>> mpegvideo_enc.o conditionnal. That should hopefully be pretty soon now.
>>     
>
> So you could change it back pretty soon: At the moment, it breaks
> -disable-mencoder for mplayer and --disable-encoders for ffmpeg.
>
>   

it builds fine here:

$./ffmpeg -i
FFmpeg version SVN-r9570, Copyright (c) 2000-2007 Fabrice Bellard, et al.
  configuration: --disable-encoders
  libavutil version: 49.4.1
  libavcodec version: 51.40.4
  libavformat version: 51.12.1
  built on Jul 10 2007 10:31:19, gcc: 3.4.6 (Gentoo 3.4.6-r1,
ssp-3.4.5-1.0, pie-8.7.9)
./ffmpeg: missing argument for option '-i'

-- 
Ben
Purple Labs S.A.
www.purplelabs.com