Merge lp://qastaging/~zhangew401/usensord/bug-1620553 into lp://qastaging/usensord

Proposed by Zhang Enwei
Status: Merged
Approved by: Ken VanDine
Approved revision: 30
Merged at revision: 28
Proposed branch: lp://qastaging/~zhangew401/usensord/bug-1620553
Merge into: lp://qastaging/usensord
Diff against target: 166 lines (+28/-28)
2 files modified
debian/changelog (+7/-0)
haptic/haptic.go (+21/-28)
To merge this branch: bzr merge lp://qastaging/~zhangew401/usensord/bug-1620553
Reviewer Review Type Date Requested Status
Pat McGowan (community) Approve
Jonas G. Drange (community) Approve
James Henstridge Pending
Samuele Pedroni Pending
Andrea Bernabei Pending
Review via email: mp+305351@code.qastaging.launchpad.net

Description of the change

fix lp:1620553,
Use another dbus connection to query senders' credentials. We cannot do writing and reading simultaneously on the same connection.

To post a comment you must log in.
Revision history for this message
Jonas G. Drange (jonas-drange) wrote :

Please add a commit message, and maybe fix that spaces diff (see comment).

Otherwise, this allows me to run the "kill_usensord" script without usensord deadlocking.

Thanks!

review: Approve
30. By Zhang Enwei

Update changelog

Revision history for this message
Pat McGowan (pat-mcgowan) :
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