Internal video player doesn't work on Wayland

[JiangHongTiao]

Hi guys,
I have downloaded subtitles&video from: https://www.youtube.com/watch?v=CeBHA2m8xm4
I have dragged and dropped automatically generated subtitles (srt) into the program. Then I have tried to load video, unfortunately app has crashed. Video is of the format.
I'm attaching bugreport and mediainfo information. Thank you for your work and time guys!

**Traceback**

File: /usr/lib/python3.6/site-packages/gaupol/agents/video.py
Line: 184
In: _on_load_video_activate

    self.player.set_path(path)

File: /usr/lib/python3.6/site-packages/gaupol/player.py
Line: 331
In: set_path

    self.set_uri(aeidon.util.path_to_uri(path))

File: /usr/lib/python3.6/site-packages/gaupol/player.py
Line: 340
In: set_uri

    self._xid = self.widget.get_window().get_xid()

AttributeError: 'GdkWaylandWindow' object has no attribute 'get_xid'

**Environment**

Platform: Linux-4.11.10-300.fc26.x86_64-x86_64-with-fedora-26-Twenty_Six
Locale: en_GB.utf_8

**Versions**

aeidon: 1.0
chardet: 2.3.0
enchant: 1.6.10
gaupol: 1.0
gstreamer: 1.12.2.0
gtk+: 3.22.17
pygobject: 3.24.1
python: 3.6.2

[otsaloma]

It seems you're running Wayland. I haven't yet tried Gaupol on Wayland myself, but it seems some changes are needed. I'll look into it when I have the time.

As a workaround, does launching Gaupol from a terminal with command GDK_BACKEND=x11 gaupol work?

[JiangHongTiao]

Hi Osmo, thanks a lot for reply and pointing me out. Yes, specifying gdk_backend Param has helped and the video is working now, sending quick screenshot. Good job! Thanks for development and your time! Juraj+ On Aug 3, 2017 19:11, "Osmo Salomaa" <[email protected]> wrote: It seems you're running Wayland. I haven't yet tried Gaupol on Wayland myself, but it seems some changes are needed. I'll look into it when I have the time. As a workaround, does launching Gaupol from a terminal with command GDK_BACKEND=x11 gaupol work? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#60 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AOHqbbMQJQW1uilo6DK_mjgRUzLlUeLuks5sUf8ygaJpZM4OnUmv> .

[JiangHongTiao]

Hi Osmo, thanks a lot for reply and pointing me out. Yes, specifying gdk_backend param has helped and the video is working now, sending quick screenshot. Good job! Thanks for development and your time! Juraj+

…

On Aug 3, 2017 19:11, "Osmo Salomaa" ***@***.***> wrote: It seems you're running Wayland. I haven't yet tried Gaupol on Wayland myself, but it seems some changes are needed. I'll look into it when I have the time. As a workaround, does launching Gaupol from a terminal with command GDK_BACKEND=x11 gaupol work? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#60 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AOHqbbMQJQW1uilo6DK_mjgRUzLlUeLuks5sUf8ygaJpZM4OnUmv> .

[otsaloma]

Could some good folks here running Wayland test the latest development version with the above potential fix? Just download, unpack and run bin/gaupol.

[JiangHongTiao]

Hi Osmo, it works for me whithout problem. The only msg in the console is: (python3:3265): Gtk-WARNING **: Allocating size to GtkApplicationWindow 0x55e6f653e2b0 without calling gtk_widget_get_preferred_width/height(). How does the code know the size to allocate? Here is at least my uname: Linux 4.13.10-200.fc26.x86_64 x86_64 x86_64 x86_64 GNU/Linux and screenshot: [image: Inline images 1] Thanks for your work! Juraj+

…

On 11 November 2017 at 02:26, Osmo Salomaa ***@***.***> wrote: Could some good folks here running Wayland test the latest development version with the above potential fix? Just download, unpack and run bin/gaupol. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#60 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AOHqbUtBDlGluLB-rf18Y0ToxmkfRg_rks5s1PexgaJpZM4OnUmv> .

[otsaloma]

Thanks, I think this is fixed now.