After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 795220 - videocontext: add wayland support to "gst.vaapi.app.Display" context
videocontext: add wayland support to "gst.vaapi.app.Display" context
Status: RESOLVED DUPLICATE of bug 705821
Product: GStreamer
Classification: Platform
Component: gstreamer-vaapi
unspecified
Other All
: Normal normal
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2018-04-13 06:33 UTC by Michael Olbrich
Modified: 2018-04-18 14:36 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
videocontext: add wayland support to "gst.vaapi.app.Display" context (3.34 KB, patch)
2018-04-13 06:33 UTC, Michael Olbrich
none Details | Review
videocontext: add wayland support to "gst.vaapi.app.Display" context (3.28 KB, patch)
2018-04-13 06:36 UTC, Michael Olbrich
none Details | Review

Description Michael Olbrich 2018-04-13 06:33:07 UTC
In bug 766704 "gst.vaapi.app.Display" was introduced with support for X11.
This adds the corresponding wayland code.
Comment 1 Michael Olbrich 2018-04-13 06:33:16 UTC
Created attachment 370880 [details] [review]
videocontext: add wayland support to "gst.vaapi.app.Display" context

Through "gst.vaapi.app.Display" context, users can set their own VADisplay
and native display of their backend.

This adds wayland support. With the 'wl-display' attribute the wayland
display pointer (struct wl_display *) can be specified.

Signed-off-by: Michael Olbrich <m.olbrich@pengutronix.de>
Comment 2 Michael Olbrich 2018-04-13 06:36:29 UTC
Created attachment 370881 [details] [review]
videocontext: add wayland support to "gst.vaapi.app.Display" context

Removed SoB.
Comment 3 Víctor Manuel Jáquez Leal 2018-04-13 07:21:21 UTC
this patch is part of bug 705821 which is still incomplete, but, yeah only this part is worth to merge
Comment 4 Michael Olbrich 2018-04-13 08:16:08 UTC
It's also useful independently to handle input events.
Comment 5 Víctor Manuel Jáquez Leal 2018-04-18 13:26:41 UTC
since the patches are exactly as Hyunjun's ones, let's mark this as duplicated and I will merged the related patches

*** This bug has been marked as a duplicate of bug 705821 ***
Comment 6 Michael Olbrich 2018-04-18 13:57:41 UTC
Ok, I think I'm missing something, because I looked at the patches from bug 705821 and I can't find the one that would allow me to set the display.
Comment 7 Víctor Manuel Jáquez Leal 2018-04-18 14:36:45 UTC
Doh! True. Hyunjun passed me his patches by email but he didn't upload them to bugzilla. I'll upload them now.