RDKOSS-808: glimagesink should support Westeros simple_shell protocol#392
Open
emutavchi wants to merge 1 commit into
Open
RDKOSS-808: glimagesink should support Westeros simple_shell protocol#392emutavchi wants to merge 1 commit into
emutavchi wants to merge 1 commit into
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates the Yocto gst-plugins-base 1.18.5 recipe to apply a new downstream patch that adds Westeros SimpleShell integration to the Wayland/EGL GL window path, allowing geometry updates via the SimpleShell protocol.
Changes:
- Add and apply a new patch to
gst-plugins-baseenablingwl_simple_shell(Westeros SimpleShell) usage in the Wayland EGL window implementation. - Extend the
waylandPACKAGECONFIG dependency list to includewesteros-simpleshell.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 5 comments.
| File | Description |
|---|---|
| recipes-multimedia/gstreamer/gstreamer1.0_1.18/gstreamer1.0-plugins-base_1.18.5.bb | Adds the new Westeros SimpleShell patch to SRC_URI and pulls westeros-simpleshell into Wayland PACKAGECONFIG deps. |
| recipes-multimedia/gstreamer/gstreamer1.0_1.18/gstreamer1.0-plugins-base/0008-GL-Wayland-Add-Westeros-SimpleShell-support.patch | Introduces Meson dependency wiring and Wayland window code changes to listen for SimpleShell surface IDs and apply geometry updates. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
b49e3ac to
3065bca
Compare
Contributor
Author
|
Here is a sample output showing the geometry change with simple shell |
3065bca to
02e5a97
Compare
And use it to change video window geometry Change-Id: Ia652ced8d014ba7b245dfb172a6b9d002dbbd649
02e5a97 to
cdca459
Compare
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
And use it to change video window geometry
Change-Id: Ia652ced8d014ba7b245dfb172a6b9d002dbbd649