Bug 192643 – [GStreamer][JHBuild] update-webkit{gtk,wpe}-libs fails with libfdk-2.0.0

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Carlos Bentzen]

Using Arch Linux, update-webkit{gtk,wpe}-libs fails if the system has libfdk-2.0.0 installed (which is the case for anybody who has gst-plugins-bad 1.14.4 in Arch).

This build issue has already been solved upstream at https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/merge_requests/77

In pacman they patched gst-plugins-bad to fix that already: https://git.archlinux.org/svntogit/packages.git/tree/trunk/PKGBUILD?h=packages/gst-plugins-bad

I'll add a patch to fix this in the GStreamer jhbuild.modules file while this is not in a release yet.

[Carlos Bentzen]

Created attachment 357193 [details]
Patch

[Philippe Normand]

Comment on attachment 357193 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=357193&action=review

> Tools/gstreamer/patches/gst-plugins-bad-0003-fdkaac-Fixes-for-compatibility-with-libfdk-aac-2.patch:36
> +From 19d34f6b5e1633d5ec4bb2832c58470f0c829cab Mon Sep 17 00:00:00 2001
> +From: "Jan Alexander Steffens (heftig)" <[e-mail]>
> +Date: Wed, 5 Dec 2018 10:10:39 +0100
> +Subject: [PATCH 2/2] fdkaacdec: Use WAV channel mapping instead of interleave
> + setting

Please use 2 files. One per patch.

[Carlos Bentzen]

(In reply to Philippe Normand from comment #2)
> Comment on attachment 357193 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=357193&action=review
> 
> > Tools/gstreamer/patches/gst-plugins-bad-0003-fdkaac-Fixes-for-compatibility-with-libfdk-aac-2.patch:36
> > +From 19d34f6b5e1633d5ec4bb2832c58470f0c829cab Mon Sep 17 00:00:00 2001
> > +From: "Jan Alexander Steffens (heftig)" <[e-mail]>
> > +Date: Wed, 5 Dec 2018 10:10:39 +0100
> > +Subject: [PATCH 2/2] fdkaacdec: Use WAV channel mapping instead of interleave
> > + setting
> 
> Please use 2 files. One per patch.

Ok.

I got the patch via https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/merge_requests/77.patch

I'll split it in two.

[Philippe Normand]

Yes but it contains 2 commits :)

[Carlos Bentzen]

Created attachment 357222 [details]
Patch

[Carlos Bentzen]

(In reply to Philippe Normand from comment #4)
> Yes but it contains 2 commits :)

Yeah indeed :) Got the patches directly from the commits now.

Thanks for noticing it!

[WebKit Commit Bot]

Comment on attachment 357222 [details]
Patch

Clearing flags on attachment: 357222

Committed r239159: <https://trac.webkit.org/changeset/239159>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Radar WebKit Bug Importer]

<rdar://problem/46694419>