Merge lp:~mterry/indicator-sound/snap-root into lp:indicator-sound
Proposed by
Michael Terry
Status: | Merged | ||||||||
---|---|---|---|---|---|---|---|---|---|
Approved by: | Charles Kerr | ||||||||
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/indicator-sound.conf.in (+1/-1) |
||||||||
To merge this branch: | bzr merge lp:~mterry/indicator-sound/snap-root | ||||||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
unity-api-1-bot | continuous-integration | Needs Fixing | |
Charles Kerr (community) | Approve | ||
Review via email: mp+308125@code.launchpad.net |
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/*.desktop.in 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.
[1] https:/
To post a comment you must log in.
LGTM