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 709905 - totem-video-thumbnailer fails on SSH with X-forwarding
totem-video-thumbnailer fails on SSH with X-forwarding
Status: RESOLVED FIXED
Product: totem
Classification: Core
Component: Thumbnailer
3.10.x
Other All
: Normal minor
: ---
Assigned To: General Totem maintainer(s)
General Totem maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2013-10-11 12:20 UTC by Philip Withnall
Modified: 2013-10-14 05:41 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
thumbnailer: Remove unused GTK+ dependency (1.97 KB, patch)
2013-10-13 11:59 UTC, Bastien Nocera
none Details | Review
thumbnailer: Remove unused GTK+ dependency (2.25 KB, patch)
2013-10-14 05:33 UTC, Bastien Nocera
committed Details | Review

Description Philip Withnall 2013-10-11 12:20:37 UTC
If I do the following:
 $ ssh -X some-hostname
 $ totem-video-thumbnailer ~/Videos/something.mpg foo.png
I get:
    ** (totem-video-thumbnailer:3597): WARNING **: Could not open X display
    Unable to init server
    couldn't parse command-line options: Cannot open display: 

This feels like something which should work.
Comment 1 Bastien Nocera 2013-10-11 12:42:02 UTC
That'd be the culprit:
g_option_context_add_group (context, gtk_get_option_group (TRUE));

Seeing as we don't have the "show the image for debug" code, we can remove that, the GTK+ include and the GTK+ linking.
Comment 2 Bastien Nocera 2013-10-13 11:59:49 UTC
Created attachment 257166 [details] [review]
thumbnailer: Remove unused GTK+ dependency
Comment 3 Bastien Nocera 2013-10-14 05:33:22 UTC
Created attachment 257205 [details] [review]
thumbnailer: Remove unused GTK+ dependency
Comment 4 Bastien Nocera 2013-10-14 05:41:05 UTC
Attachment 257205 [details] pushed as 02765d9 - thumbnailer: Remove unused GTK+ dependency