| Summary: | [Flatpak SDK] Update GStreamer and Mesa | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Philippe Normand <pnormand> | ||||||
| Component: | Tools / Tests | Assignee: | Philippe Normand <pnormand> | ||||||
| Status: | RESOLVED FIXED | ||||||||
| Severity: | Normal | CC: | aperez, lmoura | ||||||
| Priority: | P2 | ||||||||
| Version: | WebKit Nightly Build | ||||||||
| Hardware: | Unspecified | ||||||||
| OS: | Unspecified | ||||||||
| See Also: | https://bugs.webkit.org/show_bug.cgi?id=219983 | ||||||||
| Bug Depends on: | |||||||||
| Bug Blocks: | 220418 | ||||||||
| Attachments: |
|
||||||||
|
Description
Philippe Normand
2021-01-04 03:45:54 PST
Created attachment 416950 [details]
Patch
Comment on attachment 416950 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=416950&action=review Tested in a local SDK build together with GTK4's patches from bug219983 and bug219844 and there were no noticeable regressions in the tests. Other than the mismatch in the changelog, seems good. > Tools/buildstream/ChangeLog:10 > + - GStreamer 1.18.0 -> 1.18.3 Should be 1.18.2, no? Comment on attachment 416950 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=416950&action=review >> Tools/buildstream/ChangeLog:10 >> + - GStreamer 1.18.0 -> 1.18.3 > > Should be 1.18.2, no? Indeed! Created attachment 417181 [details]
Patch
Comment on attachment 417181 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=417181&action=review > Tools/buildstream/elements/sdk/mesa.bst:71 > + -Dplatforms=x11,wayland Shouldn't we leave the “drm” platform enabled here? We would need it if/when bug #217477 is taken care of to be able to use “cog --platform=drm” 🤔️ (In reply to Adrian Perez from comment #5) > Comment on attachment 417181 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=417181&action=review > > > Tools/buildstream/elements/sdk/mesa.bst:71 > > + -Dplatforms=x11,wayland > > Shouldn't we leave the “drm” platform enabled here? We would need it if/when > bug #217477 is taken care of to be able to use “cog --platform=drm” 🤔️ FWIW, we can always re-add it later on. As of https://gitlab.freedesktop.org/mesa/mesa/commit/cb728f28acbf8982d37aa3863cae58db95a439ca DRM is now unconditionally enabled. (In reply to Philippe Normand from comment #7) > As of > https://gitlab.freedesktop.org/mesa/mesa/commit/ > cb728f28acbf8982d37aa3863cae58db95a439ca DRM is now unconditionally enabled. This might not be the right commit, but I do remember the build failing because the DRM option couldn't be found anymore ;) (In reply to Philippe Normand from comment #7) > As of > https://gitlab.freedesktop.org/mesa/mesa/commit/ > cb728f28acbf8982d37aa3863cae58db95a439ca DRM is now unconditionally enabled. Great, let's land the patch, then 😼️ Landed in r271519 |