After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 616159 - Should try to get icons from EMPATHY_SRCDIR
Should try to get icons from EMPATHY_SRCDIR
Status: RESOLVED FIXED
Product: empathy
Classification: Core
Component: General
2.29.x
Other Linux
: Normal normal
: ---
Assigned To: empathy-maint
empathy-maint
Depends on:
Blocks:
 
 
Reported: 2010-04-19 10:52 UTC by Guillaume Desmottes
Modified: 2012-03-08 12:10 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
http://git.collabora.co.uk/?p=user/cassidy/empathy;a=shortlog;h=refs/heads/icons-616159 (2.38 KB, patch)
2010-04-19 12:09 UTC, Guillaume Desmottes
none Details | Review
"Build" a local copy of the icons to be used from sources (2.08 KB, patch)
2012-03-06 15:28 UTC, Emanuele Aina
none Details | Review
empathy_gtk_init: add local copy of icons to the search path (#616159) (1.12 KB, patch)
2012-03-06 15:28 UTC, Emanuele Aina
none Details | Review
Re-spin, this time generating icons only when needed. (3.06 KB, patch)
2012-03-07 12:29 UTC, Emanuele Aina
none Details | Review
Re-spin, loading from ./data/icons/local-copy (1.11 KB, patch)
2012-03-07 12:31 UTC, Emanuele Aina
none Details | Review

Description Guillaume Desmottes 2010-04-19 10:52:33 UTC
Empathy try to get its icons from PKGDATADIR/"icons". So when running from source, these icons are missing if we didn't "make install" as well.
Empathy should try to get EMPATHY_SRCDIR when the variable is set so it will be run fine without being installed.
Comment 1 Guillaume Desmottes 2010-04-19 12:09:33 UTC
Created attachment 159063 [details] [review]
http://git.collabora.co.uk/?p=user/cassidy/empathy;a=shortlog;h=refs/heads/icons-616159

 data/.gitignore                   |    1 +
 data/icons/Makefile.am            |   27 +++++++++++++++++++++++++++
 libempathy-gtk/empathy-ui-utils.c |    9 +++++++++
 3 files changed, 37 insertions(+), 0 deletions(-)
Comment 2 Guillaume Desmottes 2010-04-19 12:48:09 UTC
Note that the icons are re-installed each time "make" is executed. If an autofoo guru could improve that to only run it when icons or Makefile has been changed that would be great.
Comment 3 Guillaume Desmottes 2010-12-02 13:49:19 UTC
I rebased the branch on top of master. It still works fine but I'd prefer to fix it to to re-install all the icons each time we run "make" before merging.
Comment 4 Emanuele Aina 2012-03-06 15:28:04 UTC
Created attachment 209098 [details] [review]
"Build" a local copy of the icons to be used from sources
Comment 5 Emanuele Aina 2012-03-06 15:28:08 UTC
Created attachment 209099 [details] [review]
empathy_gtk_init: add local copy of icons to the search path (#616159)
Comment 6 Emanuele Aina 2012-03-07 12:29:17 UTC
Created attachment 209150 [details] [review]
Re-spin, this time generating icons only when needed.

With a bit of GNU Make hackery generate a rule for each icon which
will ensure that the destination stays up-to-date with the source.

I've also changed the local installation path from
./data/icons/local-copy/icons to just ./data/icons/local-copy
Comment 7 Emanuele Aina 2012-03-07 12:31:02 UTC
Created attachment 209151 [details] [review]
Re-spin, loading from ./data/icons/local-copy
Comment 8 Guillaume Desmottes 2012-03-07 13:26:13 UTC
Seems to work so if "make distcheck" still pass that's good for me.