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

Proposed by Alberto Mardegan
Status: Rejected
Rejected by: Alberto Mardegan
Proposed branch: lp:~mardy/account-plugins/remove-unused-icons
Merge into: lp: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:~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.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 :
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
1=== modified file 'data/icons/Makefile.am'
2--- data/icons/Makefile.am 2012-07-19 16:43:16 +0000
3+++ data/icons/Makefile.am 2012-07-23 10:19:22 +0000
4@@ -1,24 +1,9 @@
5 iconsdir = $(datadir)/icons/hicolor/32x32/apps
6 dist_icons_DATA = \
7- aim.png \
8 facebook.png \
9 flickr.png \
10 foursquare.png \
11- gadugadu.png \
12 google.png \
13- groupwise.png \
14 identica.png \
15- icq.png \
16- irc.png \
17- jabber.png \
18 live.png \
19- msn.png \
20- mxit.png \
21- myspace.png \
22- people-nearby.png \
23- sametime.png \
24- twitter.png \
25- yahoo.png \
26- zephyr.png
27-
28-
29+ twitter.png

Subscribers

People subscribed via source and target branches