Merge lp:~3v1n0/unity/unity7-wants-ups into lp:unity

[Marco Trevisan (Treviño)]

unity7.service: set ups and BAMF as wanted, not required

We don't want unity to crash when they do.

[Iain Lane (laney)]

01/03 09:32:31 <Laney> Trevinho: should be RequiredBy=ubuntu-session.target, no?
01/03 09:32:45 <Laney> the session itself isn't valid without unity7, but unity7 is (arguably) valid without those other things

Fix this one as you prefer

review: Approve

[Iain Lane (laney)]

This is what we have now

laney@nightingale> ls ubuntu-session.target.requires
unity7.service@
laney@nightingale> ls ubuntu-session.target.wants
unity-gtk-module.service@ unity-settings-daemon.service@ window-stack-bridge.service@

I think that's pretty good. In fact it means that you could drop u-s-d from Wants= completely in this MP, since it is responsible for starting itself up.

[lp:~3v1n0/unity/unity7-wants-ups]

4221. By Marco Trevisan (Treviño) on 2017-03-01

unity7.service: set uss, ups and BAMF as wanted, not required

We don't want unity to crash when they do.

=== modified file 'data/unity7.service.in'
--- data/unity7.service.in	2016-09-05 10:48:28 +0000
+++ data/unity7.service.in	2017-03-01 11:34:07 +0000
@@ -1,7 +1,6 @@
 [Unit]
 Description=Unity Shell v7
-Requires=unity-settings-daemon.service unity-panel-service.service bamfdaemon.service
-Wants=unity-gtk-module.service
+Wants=unity-settings-daemon.service unity-gtk-module.service unity-panel-service.service bamfdaemon.service
 After=unity-settings-daemon.service
 PartOf=graphical-session.target