Merge lp://staging/~seb128/unity-control-center/users-prompt-when-needed into lp://staging/unity-control-center

Proposed by Sebastien Bacher
Status: Merged
Approved by: Sebastien Bacher
Approved revision: 12757
Merged at revision: 12758
Proposed branch: lp://staging/~seb128/unity-control-center/users-prompt-when-needed
Merge into: lp://staging/unity-control-center
Diff against target: 18 lines (+1/-1)
1 file modified
panels/user-accounts/um-password-dialog.c (+1/-1)
To merge this branch: bzr merge lp://staging/~seb128/unity-control-center/users-prompt-when-needed
Reviewer Review Type Date Requested Status
desrt (community) Approve
PS Jenkins bot (community) continuous-integration Approve
Unity Control Center development team Pending
Review via email: mp+211596@code.staging.launchpad.net

Commit message

users: don't make accountsservice calls when not needed

Description of the change

users: don't make accountsservice calls when not needed

when changing your own password the "passwd" command is used rather than using accountsservice, in that case avoid making a call to a-s since it leads to unwanted polkit auth dialogs

the issue is mentioned on https://bugzilla.gnome.org/show_bug.cgi?id=701701 as well

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
desrt (desrt) wrote :

I recommended doing this, so +1 from me :)

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