Merge lp:~mterry/indicator-sound/snap-root into lp:indicator-sound

Proposed by Michael Terry on 2016-10-11
Status: Merged
Approved by: Charles Kerr on 2016-10-11
Approved revision: 540
Merged at revision: 542
Proposed branch: lp:~mterry/indicator-sound/snap-root
Merge into: lp:indicator-sound
Diff against target: 9 lines (+1/-1)
1 file modified
data/ (+1/-1)
To merge this branch: bzr merge lp:~mterry/indicator-sound/snap-root
Reviewer Review Type Date Requested Status
unity-api-1-bot continuous-integration Needs Fixing on 2016-10-11
Charles Kerr (community) 2016-10-11 Approve on 2016-10-11
Review via email:

Commit message

Work when inside a snap by prefixing hard coded paths with $SNAP.

Description of the change

There are also two hardcoded paths in the data/* files. But that's a static desktop file, so $SNAP isn't appropriate. I think it makes more sense to update the code that reads those files to do the prefixing.

But this, in conjunction with [1], gets indicator-sound to appear in a snappified-unity8 session.


To post a comment you must log in.
Charles Kerr (charlesk) wrote :


review: Approve
unity-api-1-bot (unity-api-1-bot) wrote :
review: Needs Fixing (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'data/'
2--- data/ 2015-12-23 14:13:32 +0000
3+++ data/ 2016-10-11 14:09:21 +0000
4@@ -6,4 +6,4 @@
5 respawn
6 respawn limit 30 10
8-exec @CMAKE_INSTALL_FULL_LIBEXECDIR@/indicator-sound/indicator-sound-service
9+exec $SNAP@CMAKE_INSTALL_FULL_LIBEXECDIR@/indicator-sound/indicator-sound-service


People subscribed via source and target branches