Hi,

On Mon, Oct 3, 2011 at 2:48 PM, Janne Grunau <[email protected]> wrote:
> On Mon, Oct 03, 2011 at 02:26:46PM -0700, Ronald S. Bultje wrote:
>>
>> >> --- a/libavcodec/x86/Makefile
>> >> +++ b/libavcodec/x86/Makefile
>> >> @@ -33,6 +33,9 @@ MMX-OBJS-$(CONFIG_ENCODERS)            += 
>> >> x86/dsputilenc_mmx.o
>> >>  YASM-OBJS-$(CONFIG_ENCODERS)           += x86/dsputilenc_yasm.o
>> >>  MMX-OBJS-$(CONFIG_GPL)                 += x86/idct_mmx.o
>> >>  MMX-OBJS-$(CONFIG_LPC)                 += x86/lpc_mmx.o
>> >> +YASM-OBJS-$(CONFIG_PRORES_DECODER)     += x86/proresdsp.o
>> >> +MMX-OBJS-$(CONFIG_PRORES_DECODER)      += x86/proresdsp-init.o          \
>> >> +                                          $(YASM-OBJS-yes)
>> >
>> > This is wrong, $(YASM-OBJS-yes) is already added to MMX-OBJS elsewhere.
>>
>> OK, but that's wrong, it should only be if CONFIG_PRORES_DECODER is
>> yes. How to do that?
>
> huh? There are other objects in $(YASM-OBJS-yes) and it won't have
> anything prores related if $(CONFIG_PRORES_DECODER) does not evaluate
> to yes.

Oh right. Let me cook up a new patch then...

Ronald
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to