Summary: | [WPE][GTK] Bump minimum versions of GLib, GTK, libsoup, ATK, GStreamer, and Cairo | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Michael Catanzaro <mcatanzaro> | ||||
Component: | WebKitGTK | Assignee: | Michael Catanzaro <mcatanzaro> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | bugs-noreply, cgarcia, mcatanzaro, pnormand | ||||
Priority: | P2 | ||||||
Version: | WebKit Nightly Build | ||||||
Hardware: | PC | ||||||
OS: | Linux | ||||||
Attachments: |
|
Description
Michael Catanzaro
2019-06-20 17:41:33 PDT
Also there were some fallthroughs in the a11y code that seemed plainly incorrect, which this fixes. Created attachment 372607 [details]
Patch
Could yopu explain in the ChangeLog where the new numbers come from? Are those the versions of the deps in Debian stretch? what about other distros? Comment on attachment 372607 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=372607&action=review > Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamerBase.cpp:-406 > -#if GST_CHECK_VERSION(1, 11, 0) > gst_structure_set(structure, "context", GST_TYPE_GL_CONTEXT, gstGLContext(), nullptr); > -#else > - gst_structure_set(structure, "context", GST_GL_TYPE_CONTEXT, gstGLContext(), nullptr); > -#endif This one needs to remain, and s/11/12/ (In reply to Carlos Garcia Campos from comment #3) > Could yopu explain in the ChangeLog where the new numbers come from? Are > those the versions of the deps in Debian stretch? what about other distros? The new versions are MIN(whatever's available in Stretch, whatever we have a CHECK_VERSION for). E.g. we have ATK_CHECK_VERSIONs looking for very recent ATKs like 2.30 or whatever, but only ATK 2.22 is available in Stretch, and the highest CHECK_VERSION we have below 2.22 is 2.16, so I set the minimum to 2.16. Other distros should all be at least as new as Stretch or they will no longer be able to update. (In reply to Philippe Normand from comment #4) > This one needs to remain, and s/11/12/ Oops, good catch! > (In reply to Philippe Normand from comment #4)
> > This one needs to remain, and s/11/12/
>
> Oops, good catch!
I also missed removing runtime checks for 1.8 and 1.10 using webkitGstCheckVersion().
Committed r246677: <https://trac.webkit.org/changeset/246677> |