Online Accounts: Account plugins

Merge lp://staging/~jbicha/account-plugins/depend-on-signon-plugin-password into lp://staging/account-plugins

  1. depend-on-signon-plugin-password
  2. Merge into trunk
Proposed by Jeremy Bícha
Status: Merged
Approved by: Alberto Mardegan
Approved revision: 169
Merged at revision: 169
Proposed branch: lp://staging/~jbicha/account-plugins/depend-on-signon-plugin-password
Merge into: lp://staging/account-plugins
Diff against target: 11 lines (+1/-0)
1 file modified
debian/control (+1/-0)
To merge this branch: bzr merge lp://staging/~jbicha/account-plugins/depend-on-signon-plugin-password
Reviewer Review Type Date Requested Status
Alberto Mardegan (community) Approve
dobey (community) Approve
Review via email: mp+304283@code.staging.launchpad.net

Commit message

* Have account-plugin-owncloud depend on signon-plugin-password

Description of the change

evolution-data-server-online-accounts no longer depends on signon-plugin-password
https://launchpad.net/ubuntu/+source/evolution-data-server/3.21.90-0ubuntu3

I'm told that account plugins using oauth don't need signon-plugin-password

data/providers/owncloud.provider.in.in
I am surprised that from my analysis OwnCloud is the only account plugin in yakkety that doesn't use oauth (or at least depend on mcp-account-manager-uoa which already depends on signon-plugin-password).

To post a comment you must log in.
lp://staging/~jbicha/account-plugins/depend-on-signon-plugin-password updated
169. By Jeremy Bícha

Have account-plugin-owncloud depend on signon-plugin-password

Revision history for this message
dobey (dobey) :
review: Approve
Revision history for this message
Alberto Mardegan (mardy) wrote :

Thanks Jeremy, this is totally correct.

review: Approve
Revision history for this message
Alberto Mardegan (mardy) wrote :

Actually, given that the original issue is with EDS, which needs it for the Yahoo plugin (Google and Windows Mail both support authenticating to IMAP via OAuth), it would be a good idea to add a "Recommends" in the account-plugin-yahoo too. Or we risk breaking EDS, if one day we decide to remove the owncloud plugin.

You are very welcome to push that change to this same branch :-)

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