Merge lp:~mterry/unity8-desktop-session/accountsservice into lp:unity8-desktop-session
| Status: | Merged |
|---|---|
| Approved by: | Christopher Townsend on 2016-11-08 |
| Approved revision: | 106 |
| Merged at revision: | 109 |
| Proposed branch: | lp:~mterry/unity8-desktop-session/accountsservice |
| Merge into: | lp:unity8-desktop-session |
| Diff against target: |
31 lines (+27/-0) 1 file modified
debian/unity8-session-snap.links (+27/-0) |
| To merge this branch: | bzr merge lp:~mterry/unity8-desktop-session/accountsservice |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Stephen M. Webb (community) | Approve on 2016-11-08 | ||
| Christopher Townsend | 2016-11-03 | Approve on 2016-11-08 | |
|
Review via email:
|
|||
Commit Message
Point the system AccountsService daemon at our registered custom properties.
Description of the Change
Add links so that the system AccountsService daemon knows about the custom properties that unity8 wants to register (but those files normally live in the snap, which the system AS daemon can't find).
So we link to it from our glue package. In an all-snaps world, we'll need to add a new interface hook for a new AS system snap. We'll figure that out later, but this is a decent fix for the classic desktop.
I've named the files so that they won't conflict if you have unity8 packages separately installed. The content of the files may conflict (like if the snap adds a new property and the debs don't) -- I'm not sure what would happen in that case. But we eventually plan for these to conflict anyway, so I don't think that's a problem.
I'm also concerned about the interaction between the snap and deb versions conflicting, but until we have a proper AccountServices snap w/ an interface, this is the best we can do.
I'll approve with my caveat mentioned:)