Merge lp:~laney/indicator-application/no-autostart-ubuntu into lp:indicator-application

Proposed by Iain Lane
Status: Merged
Approved by: Iain Lane
Approved revision: 261
Merged at revision: 261
Proposed branch: lp:~laney/indicator-application/no-autostart-ubuntu
Merge into: lp:indicator-application
Diff against target: 10 lines (+1/-0)
1 file modified
data/indicator-application.desktop.in (+1/-0)
To merge this branch: bzr merge lp:~laney/indicator-application/no-autostart-ubuntu
Reviewer Review Type Date Requested Status
Sebastien Bacher Approve
Review via email: mp+364154@code.launchpad.net

Commit message

data/indicator-application.desktop.in: Don't run in ubuntu sessions

indicator-application tries to own the org.kde.StatusNotifierWatcher name,
which the appindicator extension also wants to own. This is a race condition,
and if indicator-application wins then you get no icons in the Ubuntu session.

LP: #1819103

To post a comment you must log in.
Revision history for this message
Sebastien Bacher (seb128) wrote :

Thanks

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'data/indicator-application.desktop.in'
2--- data/indicator-application.desktop.in 2016-10-28 03:42:13 +0000
3+++ data/indicator-application.desktop.in 2019-03-08 13:13:21 +0000
4@@ -5,5 +5,6 @@
5 StartupNotify=false
6 Terminal=false
7 OnlyShowIn=Unity;GNOME;
8+NotShowIn=ubuntu;
9 NoDisplay=true
10 AutostartCondition=GNOME3 unless-session gnome

Subscribers

People subscribed via source and target branches