[FFmpeg-devel] [PATCH] Add objs to h264 parser.

[Ramiro Polla]

On Thu, Feb 11, 2010 at 10:50 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
> On Thu, Feb 11, 2010 at 08:45:34PM -0200, Ramiro Polla wrote:
>> ./configure --disable-parsers --disable-decoders --disable-encoders
>> --disable-bsfs --disable-muxers --disable-demuxers --disable-indevs
>> --disable-outdevs --disable-network --enable-parser=h264 && make
>>
>> fails to build because of undefined references.
>
>> ?Makefile | ? ?6 ++++--
>> ?1 file changed, 4 insertions(+), 2 deletions(-)
>> 83da30e5b791a57c7bcaabf34d997c8b02809af3 ?h264_parser_no_decoder.diff
>> Index: libavcodec/Makefile
>> ===================================================================
>> --- libavcodec/Makefile ? ? ? (revision 21754)
>> +++ libavcodec/Makefile ? ? ? (working copy)
>> @@ -507,8 +507,10 @@
>> ?OBJS-$(CONFIG_DVDSUB_PARSER) ? ? ? ? ? += dvdsub_parser.o
>> ?OBJS-$(CONFIG_H261_PARSER) ? ? ? ? ? ? += h261_parser.o
>> ?OBJS-$(CONFIG_H263_PARSER) ? ? ? ? ? ? += h263_parser.o
>> -OBJS-$(CONFIG_H264_PARSER) ? ? ? ? ? ? += h264_parser.o h264.o h264idct.o \
>> - ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?h264pred.o cabac.o ? ? ? ? ? ? ?\
>> +OBJS-$(CONFIG_H264_PARSER) ? ? ? ? ? ? += h264_parser.o h264.o h264idct.o h264_ps.o \
>> + ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?h264_sei.o h264_refs.o h264pred.o \
>> + ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?h264_loopfilter.o h264_direct.o \
>> + ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?cabac.o h264_cabac.o h264_cavlc.o \
>> ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?mpegvideo.o error_resilience.o
>
> iam not makefile maintainer but the parser definitly does not need the
> loopfilter code nor the idct
> if such dependancies exist they are indirect and should be specified in
> a more sane way like
> "if h264.o is included you must include h264idct.o"

Ok, then I leave this to the Makefile maintainers, or someone with a
better understanding of h264.