[FFmpeg-devel] [PATCH] Add extralibs to dependency checker

[Måns Rullgård]

Ivo <ivop at euronet.nl> writes:

> On Tuesday 18 September 2007 20:27, M?ns Rullg?rd wrote:
>> Ramiro Ribeiro Polla <ramiro at lisha.ufsc.br> writes:
>> > wrote:
>> >> Ramiro Ribeiro Polla <ramiro at lisha.ufsc.br> writes:
>> >>> @@ -750,12 +753,14 @@
>> >>>   # external libraries
>> >>>  mpeg4aac_decoder_deps="libfaad"
>> >>> +liba52bin_decoder_extralibs='$ldl'
>> >>
>> >> Hmm... there are probably enough evals to make this actually work.
>> >
>> > I don't understand what you mean. It does work though...
>>
>> I mean that the value of liba52bin_decoder_extralibs must be subjected
>> to an eval to expand the $ldl.
>
> Yes. But I'm not sure if that's the case for every possible codepath
> (i.e. combinations of configure options).

It is.

> I think it's therefore better to write:
>
> liba52bin_decoder_extralibs="$ldl"

That would certainly not work at all.

-- 
M?ns Rullg?rd
mans at mansr.com