[Buildroot] [git commit] ffmpeg: fix static linking build failure when using libavutil
Arnout Vandecappelle
arnout at mind.be
Tue Jan 29 13:39:45 UTC 2019
On 29/01/2019 13:30, Giulio Benetti wrote:
> Hi Arnout,
>
> Il 29/01/2019 10:03, Arnout Vandecappelle ha scritto:
>> Oopsie...
>>
>> On 29/01/2019 00:16, Arnout Vandecappelle (Essensium/Mind) wrote:
>>> commit:
>>> https://git.buildroot.net/buildroot/commit/?id=483db9908985d023b858c0b59d4016f9abb4b6f9
>>>
>>> branch: https://git.buildroot.net/buildroot/commit/?id=refs/heads/master
>>>
>>> If a package tries to static link with libavutil it fails due to the
>>> lack of libavutil private dependencies in libavutil.pc (-ldrm in this
>>> case).
>>>
>>> Add patch to:
>>> - Check if libdrm is present.
>>> - Add it to Libs.private: in libavutil.pc if present.
>>
>> I didn't want to push this one, it was just a test. The patch no longer applies
>> to ffmpeg 3.4.5 so this is causing autobuilder breakage. Therefore, I pushed a
>> revert commit.
>
> I've forgot it there pending on patchwork after ffmpeg bump.
I've marked it as Changes Requested now.
>
>> Giulio, FYI, upstream has added an explicit $LIBDRM but not to Libs.Private, so
>> the static linking case still breaks for the same reason. The patch just needs
>> to be rebased (and sent upstream).
>
> I've tried several times to discuss this on ffmpeg-devel:
> [1] http://ffmpeg.org/pipermail/ffmpeg-devel/2018-October/235269.html
> but they ignored my last patch:
> [2] http://ffmpeg.org/pipermail/ffmpeg-devel/2018-October/235417.html
> to fix the previous I've sent:
> [3] http://ffmpeg.org/pipermail/ffmpeg-devel/2018-October/235269.html
>
> I can't understand why they don't want to upstream my last patch[2]...
Same as everywhere: too many patches, too few reviewers. The reactions to v1
seemed to be positive. So if v2 still applies to current ffmpeg master, just
ping it; if it needs to be rebased, rebase it and resubmit.
Regards,
Arnout
>
> Anyway I'm going to submit the last one for Buildroot asap.
> I'm going to reproduce it right now.
>
> Best regards
>
More information about the buildroot
mailing list