[FFmpeg-devel] [PATCH] configure: add LIBDRM to extralibs_avutil

[James Almer]

On 9/11/2018 6:24 PM, Giulio Benetti wrote:
> Hello,
> 
> Il 11/09/2018 23:21, James Almer ha scritto:
>>> -lswscale -lavdevice -lavformat -lavcodec -lssl -lcrypto -lspeex
>>> -lmodplug -lstdc++ -lfontconfig -lfreetype -lpng16 -lexpat -ldrm -lz
>>> -pthread -latomic -lswresample  -lavutil -lm
>>
>> What i'm seeing here is that -ldrm is apparently present in
>> libavdevice.pc, but not in libavutill.pc, which shouldn't happen given
>> that it's strictly listed in the avutil_suggest list.
>>
>> What version of ffmpeg is this script using? Latest git head, an old
>> snapshot, or a release? Did this start failing recently?
> 
> It happens on version 3.4.4 and I haven't foud fixes until master(maybe
> I've missed something, I'm quite new to this).

This patch does nothing on git master. This is a fix that needs to be
applied to the 3.4 release branch, if anywhere.

> 
>> This patch is a hacky way to add the "-ldrm" ldflag to avutil_extralibs.
>> It should be added organically by configure as part of the dependency
>> checks, so this is definitely not an adequate fix.
> 
> Can you point me the right way to fix this?
> It was the only way I've found to have it working.

I guess it's acceptable in that case. The 3.4 branch had LIBRT and LIBM
hacked into extralibs_avutil, so another one wont hurt. I've pushed it
there.

Thanks.

> 
> Thanks in advance
> Giulio Benetti