Merge lp://staging/~mardy/gnome-control-center-signon/provider-data into lp://staging/gnome-control-center-signon
Proposed by
Alberto Mardegan
Status: | Merged |
---|---|
Approved by: | Alberto Mardegan |
Approved revision: | 136 |
Merged at revision: | 133 |
Proposed branch: | lp://staging/~mardy/gnome-control-center-signon/provider-data |
Merge into: | lp://staging/gnome-control-center-signon |
Diff against target: |
156 lines (+40/-12) 6 files modified
configure.ac (+3/-1) debian/control (+1/-1) libaccount-plugin/account-plugin.pc.in (+1/-0) libaccount-plugin/oauth-plugin.c (+21/-9) tests/data/MyProvider.provider (+11/-0) tests/test-account-plugin.vala (+3/-1) |
To merge this branch: | bzr merge lp://staging/~mardy/gnome-control-center-signon/provider-data |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
David King (community) | Approve | ||
PS Jenkins bot (community) | continuous-integration | Needs Fixing | |
Review via email: mp+145786@code.staging.launchpad.net |
Description of the change
Read authentication settings from .provider file
When configuring the account, also read all the authentication parameters defined in the .provider file, and merge them with those explicitly defined by the plugin binary.
Also, add a dependency on GModule, from which we are using g_module_
To post a comment you must log in.
FAILED: Continuous integration, rev:135 s-jenkins: 8080/job/ gnome-control- center- signon- ci/21/ s-jenkins: 8080/job/ gnome-control- center- signon- ci/./build= pbuilder, distribution= raring, flavor= amd64/21/ console s-jenkins: 8080/job/ gnome-control- center- signon- ci/./build= pbuilder, distribution= raring, flavor= i386/21/ console
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild: s-jenkins: 8080/job/ gnome-control- center- signon- ci/21// rebuild/?
http://