[Buildroot] [PATCH v1] package/wpewebkit: fix compile without video support

Adrian Perez de Castro aperez at igalia.com
Fri Nov 13 21:50:46 UTC 2020


Hello there!

On Thu, 12 Nov 2020 22:37:34 +0100 Peter Seiderer <ps.report at gmx.net> wrote:
> Hello Peter,
> 
> On Thu, 12 Nov 2020 22:20:26 +0100, Peter Korsgaard <peter at korsgaard.com> wrote:
> 
> > >>>>> "Peter" == Peter Seiderer <ps.report at gmx.net> writes:  
> > 
> >  > Fixes:
> >  >   - https://bugs.busybox.net/show_bug.cgi?id=13306  
> > 
> >  >       .../wpewebkit-2.30.2/Source/WebKit/WebProcess/InjectedBundle/InjectedBundle.cpp:242:30: error: ‘class WebCore::Settings’ has no member named ‘setGenericCueAPIEnabled’; did you mean ‘setBeaconAPIEnabled’?
> >  >  page-> settings().setGenericCueAPIEnabled(enabled);
> >  >                                     ^~~~~~~~~~~~~~~~~~~~~~~
> >  >                                     setBeaconAPIEnabled  
> > 
> >  > Signed-off-by: Peter Seiderer <ps.report at gmx.net>  
> > 
> > Committed, thanks. Don't forget to submit the patch upstream.
> > 
> 
> Patch seems upstream already superseded by [1] 'Remove remaining alternative
> preference setting mechanisms from LayoutTests' (removing the code)...

I looked a bit into backporting this patch (and a few dependent ones) into the
stable release branch and it got complicated quickly. I think that it may be
better to have the fix that just uses the ENABLE(VIDEO) guards merged in the
release branch.

I have just created an issue in WebKit's Bugzilla:

  https://bugs.webkit.org/show_bug.cgi?id=218921

and I will have a fix done which will be included in version 2.30.3, which
is to be released soon and will also include security fixes. Maybe we would
prefer to wait until that version is released and update Buildroot to it
before ending the release candidate period. WDYT?

—Adrián

> [1] https://trac.webkit.org/changeset?reponame=webkit&new=269236%40trunk%2FSource%2FWebKit%2FWebProcess%2FInjectedBundle%2FInjectedBundle.cpp&old=269111%40trunk%2FSource%2FWebKit%2FWebProcess%2FInjectedBundle%2FInjectedBundle.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 195 bytes
Desc: not available
URL: <http://lists.buildroot.org/pipermail/buildroot/attachments/20201113/5521474b/attachment-0001.asc>


More information about the buildroot mailing list