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 651960 - Vino server can't be activated for usage in telepathy when it's not globally enabled
Vino server can't be activated for usage in telepathy when it's not globally ...
Status: RESOLVED FIXED
Product: vino
Classification: Applications
Component: Server
3.0.x
Other Linux
: Normal normal
: ---
Assigned To: Vino Maintainer(s)
Vino Maintainer(s)
: 666205 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2011-06-06 00:57 UTC by Sjoerd Simons
Modified: 2012-05-15 15:55 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
add vino_dbus_listener_get_server() (1.37 KB, patch)
2012-04-24 10:25 UTC, Guillaume Desmottes
committed Details | Review
Add VinoServer:reject-incoming property (4.64 KB, patch)
2012-04-24 10:25 UTC, Guillaume Desmottes
committed Details | Review
Add --tube arg launch Vino in 'tube only' mode (5.13 KB, patch)
2012-04-24 10:25 UTC, Guillaume Desmottes
committed Details | Review
vino-tube-server-managers: add tube-disconnected signal (1.80 KB, patch)
2012-04-24 10:25 UTC, Guillaume Desmottes
committed Details | Review
dbus-listener: relay the tube-disconnected signal (2.00 KB, patch)
2012-04-24 10:25 UTC, Guillaume Desmottes
committed Details | Review
Exit when tube is closed if run in tube mode (1.56 KB, patch)
2012-04-24 10:25 UTC, Guillaume Desmottes
committed Details | Review

Description Sjoerd Simons 2011-06-06 00:57:39 UTC
When a user shares their desktop via empathy, vino-server will be activated by telepathy over D-Bus to provide the necessary service. Unfortunately if it's not globally enabled it will just exit right away, causing the desktop sharing to instantly fail.

Instead vino-server should startup normally, but not start any servers when not globally enabled. Optionally automatically exitting after various second. This way Telepathy can start it over dbus for its purposes.
Comment 1 David King 2011-06-06 05:28:44 UTC
I am guessing that this behaviour change occurred in the port to GSettings, as the server now exits immediately upon being disabled:

http://git.gnome.org/browse/vino/commit/?id=13a2c919405e802fa86efa9a525fc204b9853f93

which managed to workaround a bug:

https://bugzilla.gnome.org/show_bug.cgi?id=599104

I'll search for the real source of the bug, and then the change can be reverted.
Comment 2 Guillaume Desmottes 2012-04-24 08:29:08 UTC
Thanks for the bug report. This particular bug has already been reported into our bug tracking system, but please feel free to report any further bugs you find.

*** This bug has been marked as a duplicate of bug 666205 ***
Comment 3 Guillaume Desmottes 2012-04-24 10:25:44 UTC
Created attachment 212677 [details] [review]
add vino_dbus_listener_get_server()
Comment 4 Guillaume Desmottes 2012-04-24 10:25:47 UTC
Created attachment 212678 [details] [review]
Add VinoServer:reject-incoming property
Comment 5 Guillaume Desmottes 2012-04-24 10:25:50 UTC
Created attachment 212679 [details] [review]
Add --tube arg launch Vino in 'tube only' mode

This fix "Share my Desktop" feature when screen sharing has been disabled in
the control center.

This regression has been introduced when Vino switched to GSettings as it
started exiting right away when desktop sharing was disabled.
Comment 6 Guillaume Desmottes 2012-04-24 10:25:52 UTC
Created attachment 212680 [details] [review]
vino-tube-server-managers: add tube-disconnected signal
Comment 7 Guillaume Desmottes 2012-04-24 10:25:56 UTC
Created attachment 212681 [details] [review]
dbus-listener: relay the tube-disconnected signal
Comment 8 Guillaume Desmottes 2012-04-24 10:25:59 UTC
Created attachment 212682 [details] [review]
Exit when tube is closed if run in tube mode
Comment 9 Guillaume Desmottes 2012-04-24 10:28:07 UTC
*** Bug 666205 has been marked as a duplicate of this bug. ***
Comment 10 David King 2012-04-25 06:44:14 UTC
Comment on attachment 212677 [details] [review]
add vino_dbus_listener_get_server()

Pushed to master as commit 5d9f371f9ea016b3e7033fe24912c21f39a61c3c.
Comment 11 David King 2012-04-25 06:44:37 UTC
Comment on attachment 212678 [details] [review]
Add VinoServer:reject-incoming property

Pushed to master as commit 83b6f3e602088a64b3a67e5bfebd87472889b141.
Comment 12 David King 2012-04-25 06:45:56 UTC
Comment on attachment 212679 [details] [review]
Add --tube arg launch Vino in 'tube only' mode

Pushed to master as commit be8bc1d91cad4635a7558cf34a48ae369c9ec830. I changed the argument description.
Comment 13 David King 2012-04-25 06:48:02 UTC
Comment on attachment 212680 [details] [review]
vino-tube-server-managers: add tube-disconnected signal

Pushed to master as commit 0d8ff9819fd0bdb86007480972fce3940eb8e8bc.
Comment 14 David King 2012-04-25 06:48:34 UTC
Comment on attachment 212681 [details] [review]
dbus-listener: relay the tube-disconnected signal

Pushed to master as commit 970207fdc820bb11b4e1ac23ac16b5ef37cf7d0d.
Comment 15 David King 2012-04-25 06:49:06 UTC
Comment on attachment 212682 [details] [review]
Exit when tube is closed if run in tube mode

Pushed to master as commit b2635742133a338e937387f0d7336bc8ac86f737, thanks!
Comment 16 Guillaume Desmottes 2012-04-25 07:17:03 UTC
Too risky for 3.2 I guess?
Comment 17 Guillaume Desmottes 2012-04-25 07:19:56 UTC
3.4 I meant
Comment 18 David King 2012-05-07 23:35:39 UTC
This is 3.6 only for the moment, as a new translatable string is added.
Comment 19 Guillaume Desmottes 2012-05-15 15:55:30 UTC
(In reply to comment #18)
> This is 3.6 only for the moment, as a new translatable string is added.

This string is just displayed in --help; we could just no translate it in stable.