[gnome-shell/benzea/systemd-3.36-units: 160/160] data: Create generic org.gnome.Shell.target

[Benjamin Berg <bberg src gnome org>]

commit 95080adde8a9a520acf785a55bfb4372993f7781
Author: Benjamin Berg <bberg redhat com>
Date:   Tue Jun 9 13:01:39 2020 +0200

    data: Create generic org.gnome.Shell.target
    
    Move the GNOME shell service file adapation for x11/wayland into the
    target/service files. This means that the session definition can simply
    pull in org.gnome.Shell.target, without having to care about whether it
    is starting an X11 or wayland session.
    
    Note that this currently requires fork'ing to do the test. This will
    however not be needed in the long term when ConditionEnvironment becomes
    available (see https://github.com/systemd/systemd/pull/15817).
    
    We technically do not need to use template units. But doing so means
    that the unit can be translated to the app id more easily (though it is
    not yet completely clear how this should look like in the long term).
    
    https://gitlab.gnome.org/GNOME/gnome-shell/merge_requests/895

 data/meson.build                                        | 11 +++++------
 data/org.gnome.Shell-x11.target                         | 10 ----------
 ...nome.Shell-wayland.target => org.gnome.Shell.target} |  4 ++--
 ...nd.service.in => org gnome Shell wayland service in} |  4 ++++
 ...ll-x11.service.in => org gnome Shell x11 service in} | 17 +++++++++++++----
 5 files changed, 24 insertions(+), 22 deletions(-)
---
diff --git a/data/meson.build b/data/meson.build
index da9d704f75..25a81422f8 100644
--- a/data/meson.build
+++ b/data/meson.build
@@ -101,21 +101,20 @@ if have_systemd
   unitconf.set('bindir', bindir)
 
   configure_file(
-    input: 'org.gnome.Shell-x11.service.in',
-    output: 'org.gnome.Shell-x11.service',
+    input: 'org gnome Shell x11 service in',
+    output: 'org.gnome.Shell@x11.service',
     configuration: unitconf,
     install_dir: systemduserunitdir
   )
 
   configure_file(
-    input: 'org.gnome.Shell-wayland.service.in',
-    output: 'org.gnome.Shell-wayland.service',
+    input: 'org gnome Shell wayland service in',
+    output: 'org.gnome.Shell@wayland.service',
     configuration: unitconf,
     install_dir: systemduserunitdir
   )
 
-  units = files('org.gnome.Shell-x11.target',
-                'org.gnome.Shell-wayland.target',
+  units = files('org.gnome.Shell.target',
                 'org.gnome.Shell-disable-extensions.service')
 
   install_data(units, install_dir: systemduserunitdir)
diff --git a/data/org.gnome.Shell-wayland.target b/data/org.gnome.Shell.target
similarity index 70%
rename from data/org.gnome.Shell-wayland.target
rename to data/org.gnome.Shell.target
index 2cb2c2e60e..358189f26f 100644
--- a/data/org.gnome.Shell-wayland.target
+++ b/data/org.gnome.Shell.target
@@ -6,5 +6,5 @@ Requisite=gnome-session-initialized.target
 PartOf=gnome-session-initialized.target
 Before=gnome-session-initialized.target
 
-Requires=org.gnome.Shell-wayland.service
-After=org.gnome.Shell-wayland.service
+Wants=org.gnome.Shell@wayland.service
+Wants=org.gnome.Shell@x11.service
diff --git a/data/org.gnome.Shell-wayland.service.in b/data/org gnome Shell wayland service in
similarity index 79%
rename from data/org.gnome.Shell-wayland.service.in
rename to data/org gnome Shell wayland service in
index 98c8aa123a..7f59d46197 100644
--- a/data/org.gnome.Shell-wayland.service.in
+++ b/data/org gnome Shell wayland service in
@@ -15,6 +15,10 @@ Before=gnome-session-initialized.target
 
 [Service]
 Type=notify
+# NOTE: This can be replaced with ConditionEnvironment=XDG_SESSION_TYPE=%I
+#       with systemd >= 245
+# Only start if the template instance matches the session type
+ExecCondition=/bin/sh -c 'test "$XDG_SESSION_TYPE" = "%I" || exit 2'
 ExecStart=@bindir@/gnome-shell
 
 # unset some environment variables that were set by the shell and won't work now that the shell is gone
diff --git a/data/org.gnome.Shell-x11.service.in b/data/org gnome Shell x11 service in
similarity index 69%
rename from data/org.gnome.Shell-x11.service.in
rename to data/org gnome Shell x11 service in
index f3c6b64773..d74bcd6436 100644
--- a/data/org.gnome.Shell-x11.service.in
+++ b/data/org gnome Shell x11 service in
@@ -18,13 +18,22 @@ StartLimitIntervalSec=15s
 StartLimitBurst=3
 
 [Service]
-Type=notify
-ExecStart=@bindir@/gnome-shell
-# Exit code 1 means we are probably *not* dealing with an extension failure
-SuccessExitStatus=1
 # On X11 we want to restart on-success (Alt+F2 + r) and on-failure.
 Restart=always
 # Do not wait before restarting the shell
 RestartSec=0ms
+
+Type=notify
+# NOTE: This can be replaced with ConditionEnvironment=XDG_SESSION_TYPE=%I
+#       with systemd >= 245. Also, the current solution is kind of painful
+#       as systemd will try to restart before giving up.
+# Only start if the template instance matches the session type.
+ExecCondition=/bin/sh -c 'test "$XDG_SESSION_TYPE" = "%I" || exit 2'
+ExecStart=@bindir@/gnome-shell
+
+# On X11 we do not need to unset any variables
+
+# Exit code 1 means we are probably *not* dealing with an extension failure
+SuccessExitStatus=1
 # Kill any stubborn child processes after this long
 TimeoutStopSec=5