[FFmpeg-devel] [PATCH] configure: fix configure check for lilv-0

Jan Ekström jeebjp at gmail.com
Sun May 13 19:17:38 EEST 2018


On Sun, May 13, 2018 at 7:12 PM, Paul B Mahol <onemda at gmail.com> wrote:
> On 5/13/18, Jan Ekstroem <jeebjp at gmail.com> wrote:
>> On Sun, May 13, 2018 at 6:52 PM, Paul B Mahol <onemda at gmail.com> wrote:
>>> On 5/13/18, Niklas Haas <ffmpeg at haasn.xyz> wrote:
>>>> From: Niklas Haas <git at haasn.xyz>
>>>>
>>>> This should be included as `<lilv/lilv.h>`, same as is done in af_lv2.c.
>>>> Forcing the extra lilv-0 breaks platforms where the include dir is
>>>> `/usr/include/lilv/lilv.h` rather than
>>>> `/usr/include/lilv-0/lilv/lilv.h`.
>>>>
>>>> The new include path works for both, because the `pkg-config --cflags`
>>>> includes `-I/usr/include/lilv-0`.
>>>> ---
>>>>  configure | 2 +-
>>>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>>>
>>>
>>> lgtm
>>
>> Thank you. Pushed in.
>>
>> Should this be back-ported to release/4.0 as well?
>
> If you care.

You may say that I do not care, but I have this still vivid in my
memory and I would guess you would like your filter to build better in
releases as well.


Best regards,
Jan


More information about the ffmpeg-devel mailing list