[FFmpeg-cvslog] r10808 - in trunk: libavcodec/allcodecs.c libavformat/allformats.c

[JonY]

Aurelien Jacobs wrote:
> On Sat, 20 Oct 2007 16:36:28 +0800
> JonY <10walls at gmail.com> wrote:
> 
>> aurel wrote:
>>> Author: aurel
>>> Date: Fri Oct 19 01:59:31 2007
>>> New Revision: 10808
>>>
>>> Log:
>>> ensure all compiled in codec/muxer are registered
>>>
>>> Modified:
>>>    trunk/libavcodec/allcodecs.c
>>>    trunk/libavformat/allformats.c
>>>
>>> Modified: trunk/libavcodec/allcodecs.c
>>> ==============================================================================
>>> --- trunk/libavcodec/allcodecs.c	(original)
>>> +++ trunk/libavcodec/allcodecs.c	Fri Oct 19 01:59:31 2007
>>> @@ -190,8 +190,7 @@ void avcodec_register_all(void)
>>>      REGISTER_ENCDEC  (LIBGSM_MS, libgsm_ms);
>>>      REGISTER_ENCODER (LIBMP3LAME, libmp3lame);
>>>      REGISTER_ENCODER (LIBTHEORA, libtheora);
>>> -    if (!ENABLE_VORBIS_ENCODER)  REGISTER_ENCODER (LIBVORBIS, libvorbis);
>>> -    if (!ENABLE_VORBIS_DECODER)  REGISTER_DECODER (LIBVORBIS, libvorbis);
>>> +    REGISTER_ENCDEC  (LIBVORBIS, libvorbis);
>>>      REGISTER_DECODER (MACE3, mace3);
>>>      REGISTER_DECODER (MACE6, mace6);
>>>      REGISTER_ENCDEC  (MP2, mp2);
>>>
>>> Modified: trunk/libavformat/allformats.c
>>> ==============================================================================
>>> --- trunk/libavformat/allformats.c	(original)
>>> +++ trunk/libavformat/allformats.c	Fri Oct 19 01:59:31 2007
>>> @@ -94,8 +94,7 @@ void av_register_all(void)
>>>      REGISTER_DEMUXER  (INGENIENT, ingenient);
>>>      REGISTER_DEMUXER  (IPMOVIE, ipmovie);
>>>      REGISTER_DEMUXER  (LIBDC1394, libdc1394);
>>> -    if (!ENABLE_NUT_DEMUXER) REGISTER_DEMUXER (LIBNUT, libnut);
>>> -    if (!ENABLE_NUT_MUXER)   REGISTER_MUXER   (LIBNUT, libnut);
>>> +    REGISTER_MUXDEMUX (LIBNUT, libnut);
>>>      REGISTER_MUXDEMUX (M4V, m4v);
>>>      REGISTER_MUXDEMUX (MATROSKA, matroska);
>>>      REGISTER_MUXER    (MATROSKA_AUDIO, matroska_audio);
>>> _______________________________________________
>>> ffmpeg-cvslog mailing list
>>> ffmpeg-cvslog at mplayerhq.hu
>>> http://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-cvslog
>>>
>> Hi,
>>
>> Changes to allcodecs.c section seems to break --disable-tremor-internal 
>> in MPlayer configure.
> 
> Then the bug is in mplayer, not in ffmpeg !
> Anyway, I've just tried to build mplayer without any problem:
> 
> $ cd mplayer
> $ svn up
> $ make distclean
> $ ./configure --disable-tremor-internal
> $ make
> => build sucessfull
> 
> Aurel
> _______________________________________________
> ffmpeg-cvslog mailing list
> ffmpeg-cvslog at mplayerhq.hu
> http://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-cvslog
> 

Hi, sorry for bugging you again.

I did a fresh check out, and still no luck. I found out that build only 
fails only if libvorbis is used instead of tremor.

I tested with these:
./configure --enable-tremor-internal --disable-tremor-external
./configure --disable-tremor-internal --enable-tremor-external
./configure --disable-tremor-internal --disable-tremor-external

The first case succeeded. Ditto for second case but external tremor 
needed. The third case uses libvorbis for OggVorbis, fails during linking.

Is there anyway I can help?