Merge lp:~saviq/unity8-desktop-session/start-indicators-lp1308891 into lp:unity8-desktop-session

Proposed by Michał Sawicz
Status: Merged
Approved by: Nick Dedekind
Approved revision: 47
Merged at revision: 47
Proposed branch: lp:~saviq/unity8-desktop-session/start-indicators-lp1308891
Merge into: lp:unity8-desktop-session
Diff against target: 19 lines (+2/-1)
1 file modified
data/unity8-mir.conf.in (+2/-1)
To merge this branch: bzr merge lp:~saviq/unity8-desktop-session/start-indicators-lp1308891
Reviewer Review Type Date Requested Status
Nick Dedekind (community) Approve
Review via email: mp+216261@code.launchpad.net

Commit message

Emit indicator-services-start upstart event in pre-start.

To post a comment you must log in.
Revision history for this message
Nick Dedekind (nick-dedekind) wrote :

LGTM.

review: Approve
48. By Michał Sawicz

Add missing "emits" entry.

49. By Michał Sawicz

Emit the indicators signal in the background, to not block unity8 starting.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'data/unity8-mir.conf.in'
2--- data/unity8-mir.conf.in 2014-04-02 20:27:10 +0000
3+++ data/unity8-mir.conf.in 2014-04-17 12:07:10 +0000
4@@ -1,6 +1,6 @@
5 description "Unity Shell v8 on Mir"
6
7-emits scope-ui-starting
8+emits scope-ui-starting indicator-services-start
9
10 start on xsession SESSION=unity8-mir and started dbus
11 stop on desktop-end
12@@ -47,6 +47,7 @@
13 fi
14
15 initctl emit scope-ui-starting
16+ initctl emit indicator-services-start &
17 end script
18
19 exec ${BINARY:-unity8} $ARGS

Subscribers

People subscribed via source and target branches