Re: [vdagent-linux v2] spice-vdagent.desktop: Change autostart to WindowManager stage

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 9/7/18 11:10 AM, Victor Toso wrote:
From: Victor Toso <me@xxxxxxxxxxxxxx>

It is possible that g_get_user_special_dir() return NULL for
G_USER_DIRECTORY_DESKTOP or G_USER_DIRECTORY_DOWNLOAD; This is a sure
thing to happen in case xdg-user-dirs hasn't been run yet.

Keeping spice-vdagent to run on Initialization stage of
X-GNOME-Autostart-Phase could trigger a race with xdg-user-dirs that
also runs on Initialization, which is the earliest stage possible.

Moving to spice-vdagent to run WindowManager stage avoids the race
with xdg-user-dirs and still runs before any graphical application.

Tested also moving to Desktop stage, which is 4th stage (out of 5) and
worked without apparent drawbacks.

Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1623947
Signed-off-by: Victor Toso <victortoso@xxxxxxxxxx>

Acked-by: Uri Lublin <uril@xxxxxxxxxx>

---
  data/spice-vdagent.desktop | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/data/spice-vdagent.desktop b/data/spice-vdagent.desktop
index 3f91af4..4424061 100644
--- a/data/spice-vdagent.desktop
+++ b/data/spice-vdagent.desktop
@@ -5,5 +5,5 @@ Exec=/usr/bin/spice-vdagent
  Terminal=false
  Type=Application
  Categories=
-X-GNOME-Autostart-Phase=Initialization
+X-GNOME-Autostart-Phase=WindowManager
  NoDisplay=true


_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux Virtualization]     [Linux Virtualization]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]