Merge lp://qastaging/~mardy/gnome-control-center-signon/reuse-plugin into lp://qastaging/gnome-control-center-signon

Proposed by Alberto Mardegan
Status: Merged
Approved by: Alberto Mardegan
Approved revision: 129
Merged at revision: 128
Proposed branch: lp://qastaging/~mardy/gnome-control-center-signon/reuse-plugin
Merge into: lp://qastaging/gnome-control-center-signon
Diff against target: 81 lines (+19/-3)
3 files modified
configure.ac (+1/-1)
debian/control (+1/-1)
libaccount-plugin/client.c (+17/-1)
To merge this branch: bzr merge lp://qastaging/~mardy/gnome-control-center-signon/reuse-plugin
Reviewer Review Type Date Requested Status
David King (community) Approve
PS Jenkins bot (community) continuous-integration Needs Fixing
Review via email: mp+142482@code.qastaging.launchpad.net

Description of the change

libaccount-plugin: load plugin as specified by provider XML file

The latest libaccounts-glib allows provider XML files to specify the plugin
name; and now libaccount-plugin will use that information to find which plugin
to load, falling back to the provider name when the plugin name is absent.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
David King (amigadave) wrote :

Missing a bump for the libaccounts-glib dependency in configure.ac. Otherwise, looks fine.

review: Needs Fixing
129. By Alberto Mardegan

Update libaccounts-glib required version in configure.ac

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
David King (amigadave) :
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