Attachment 114096 Details for Bug 89472 – systemd: make sure to only start plymouth is started

[patch] systemd: make sure to only start plymouth is started

systemd-make-sure-to-only-start-plymouth-is-starte.patch (text/plain), 1.65 KB, created by Ray Strode [halfline] on 2015-03-06 20:38:37 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Ray Strode [halfline]

Created: 2015-03-06 20:38:37 UTC

Size: 1.65 KB

patch

obsolete

>From 02c6d80d425361deabbc9bff2224cdd773f785e5 Mon Sep 17 00:00:00 2001
>From: Ray Strode <rstrode@redhat.com>
>Date: Fri, 6 Mar 2015 15:36:29 -0500
>Subject: [PATCH] systemd: make sure to only start plymouth is started
>
>This commit makes sure gdm doesn't start in parallel with plymouth,
>which could lead to plymouth activating after X is started.
>
>Furthermore, GDM is replacing getty@tty1.service and
>plymouth-quit.service, so it needs to have the same dependencies
>as them.
>
>https://bugs.freedesktop.org/show_bug.cgi?id=89472
>---
> data/gdm.service.in | 13 +++++++++++--
> 1 file changed, 11 insertions(+), 2 deletions(-)
>
>diff --git a/data/gdm.service.in b/data/gdm.service.in
>index ed74f21..79988bb 100644
>--- a/data/gdm.service.in
>+++ b/data/gdm.service.in
>@@ -1,16 +1,25 @@
> [Unit]
> Description=GNOME Display Manager
>-Conflicts=getty@tty@GDM_INITIAL_VT@.service @PLYMOUTH_QUIT_SERVICE@
>-After=systemd-user-sessions.service getty@tty@GDM_INITIAL_VT@.service @PLYMOUTH_QUIT_SERVICE@
> 
>+# replaces the getty
>+Conflicts=getty@tty@GDM_INITIAL_VT@.service
>+After=getty@tty@GDM_INITIAL_VT@.service
>+
>+# replaces plymouth-quit since it quits plymouth on its own
>+Conflicts=@PLYMOUTH_QUIT_SERVICE@
>+After=@PLYMOUTH_QUIT_SERVICE@
>+
>+# Needs all the dependencies of the services it's replacing
>+# pulled from getty@.service and @PLYMOUTH_QUIT_SERVICE@
>+After=rc-local.service plymouth-start.service plymouth-quit-wait.service systemd-user-sessions.service
> [Service]
> ExecStart=@sbindir@/gdm
> Restart=always
> IgnoreSIGPIPE=no
> BusName=org.gnome.DisplayManager
> StandardOutput=syslog
> StandardError=inherit
> EnvironmentFile=@LANG_CONFIG_FILE@
> 
> [Install]
> Alias=display-manager.service
>-- 
>2.1.0

Actions: View | Splinter Review

Attachments on bug 89472: 114091 | 114096

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.