Online Accounts: Account plugins

Merge lp://staging/~mardy/account-plugins/remove-unused-icons into lp://staging/account-plugins

  1. remove-unused-icons
  2. Merge into trunk
Proposed by Alberto Mardegan
Status: Rejected
Rejected by: Alberto Mardegan
Proposed branch: lp://staging/~mardy/account-plugins/remove-unused-icons
Merge into: lp://staging/account-plugins
Diff against target: 29 lines (+1/-16)
1 file modified
data/icons/Makefile.am (+1/-16)
To merge this branch: bzr merge lp://staging/~mardy/account-plugins/remove-unused-icons
Reviewer Review Type Date Requested Status
Alberto Mardegan (community) Disapprove
jenkins (community) continuous-integration Approve
Xavier Claessens Pending
Review via email: mp+116233@code.staging.launchpad.net

Description of the change

Do not install unused icons

There is no point in installing this icons, without an associated plugin.

To post a comment you must log in.
Revision history for this message
jenkins (martin-mrazik+qa) wrote :

PASSED: Continuous integration, rev:54
http://s-jenkins:8080/job/online-accounts-account-plugins-ci/2/
Executed test runs:
    SUCCESS: http://s-jenkins:8080/job/online-accounts-account-plugins-ci/./label=precise-clean-i386/2/console

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

Discarding: instead of removing the icons, we'll install them into a package for Empathy's use.

review: Disapprove

Unmerged revisions

54. By Alberto Mardegan

Do not install unused icons

There is no point in installing this icons, without an associated plugin.

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