Unity

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

unity7-wants-ups
Merge into trunk

Proposed by Marco Trevisan (Treviño) on 2017-02-28

Status:	Merged
Approved by:	Andrea Azzarone on 2017-03-01
Approved revision:	no longer in the source branch.
Merged at revision:	4225
Proposed branch:	lp://staging/~3v1n0/unity/unity7-wants-ups
Merge into:	lp://staging/unity
Diff against target:	11 lines (+1/-2) 1 file modified data/unity7.service.in (+1/-2)
To merge this branch:	bzr merge lp://staging/~3v1n0/unity/unity7-wants-ups
Related bugs:	Link a bug report

Reviewer	Date Requested	Status
Iain Lane (community)		Approve on 2017-03-01
Unity Team	2017-02-28	Pending
Review via email: mp+318517@code.staging.launchpad.net

Commit message

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

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

Revision history for this message

Iain Lane (laney) wrote on 2017-03-01:

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

Revision history for this message

Iain Lane (laney) wrote on 2017-03-01:

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.