Merge lp://qastaging/~zhangew401/usensord/fix-lp-1628042 into lp://qastaging/usensord

Proposed by Zhang Enwei
Status: Merged
Approved by: Pat McGowan
Approved revision: 30
Merged at revision: 30
Proposed branch: lp://qastaging/~zhangew401/usensord/fix-lp-1628042
Merge into: lp://qastaging/usensord
Diff against target: 54 lines (+14/-5)
2 files modified
debian/changelog (+8/-0)
haptic/haptic.go (+6/-5)
To merge this branch: bzr merge lp://qastaging/~zhangew401/usensord/fix-lp-1628042
Reviewer Review Type Date Requested Status
Tiago Salem Herrmann (community) Approve
Pat McGowan (community) Approve
Review via email: mp+306985@code.qastaging.launchpad.net

Description of the change

fix lp:1628042, add telephony-service-approver and telephony-service-indicator as privileged caller to usensord

To post a comment you must log in.
Revision history for this message
Pat McGowan (pat-mcgowan) wrote :

Looks good, the code makes sense and the new package in ticket 2014 works per the test plan

review: Approve
Revision history for this message
Tiago Salem Herrmann (tiagosh) wrote :

looks good.
I am no expert in usensord, but adding telephony-service-* as privileged processes looks correct to me.
Thanks Enwei!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches