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 598833 - Do not ship dropped evo_mail_notifier.png
Do not ship dropped evo_mail_notifier.png
Status: RESOLVED FIXED
Product: evolution
Classification: Applications
Component: general
2.28.x (obsolete)
Other Linux
: Normal normal
: ---
Assigned To: Evolution Shell Maintainers Team
Evolution QA team
: 607052 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2009-10-18 10:52 UTC by Yves-Alexis Perez
Modified: 2010-09-30 06:46 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Don't ship evo_mail_notifier.png (773 bytes, patch)
2009-10-18 18:33 UTC, Yves-Alexis Perez
committed Details | Review

Description Yves-Alexis Perez 2009-10-18 10:52:18 UTC
Hey,

it seems that the evo_mail_notifier.png file is missing from the tarball, breaking the documentation:

corsac@hidalgo: tar tvf evolution_2.28.0.orig.tar.gz|grep notifier
-rw-r--r-- 1002/100       7986 2009-09-21 12:30 evolution-2.28.0/help/fr/figures/evo_mail_notifier.png
-rw-r--r-- 1002/100       4452 2009-09-21 12:30 evolution-2.28.0/help/C/figures/evo_mail_notifier.png
-rw-r--r-- 1002/100       4452 2009-09-21 12:30 evolution-2.28.0/help/eu/figures/evo_mail_notifier.png
corsac@hidalgo: tar tvf evolution_2.28.1.orig.tar.gz|grep notifier
-rw-r--r-- 500/500        4452 2009-10-18 04:25 evolution-2.28.1/help/eu/figures/evo_mail_notifier.png
-rw-r--r-- 500/500        7986 2009-10-18 04:25 evolution-2.28.1/help/fr/figures/evo_mail_notifier.png

It might not be too late to fix the tarball before the “official” release tomorrow?
Comment 1 André Klapper 2009-10-18 12:49:50 UTC
Please always provide the exact error message and explain why you think that this is an issue at all.
That .png file is unsed in evolution.xml.
Comment 2 Yves-Alexis Perez 2009-10-18 15:27:17 UTC
(In reply to comment #1)
> Please always provide the exact error message and explain why you think that
> this is an issue at all.
> That .png file is unsed in evolution.xml.

Well, the build system seems to make symlinks for “missing” png, which means we end up with a symlink from C/figures/evo_mail_notifier.png to ../../C/figures/evo_mail_notifier.png which doesn't really make sense.

And, by the way, I find it weird that the documentation is changed in a stable release, and not changed for every translation (fr and eu still have the file).
Comment 3 Yves-Alexis Perez 2009-10-18 18:33:04 UTC
Ok, I saw that quite some png were dropped in these release. I guess this one should then be dropped as well. Attached patch should do that.
Comment 4 Yves-Alexis Perez 2009-10-18 18:33:39 UTC
Created attachment 145742 [details] [review]
Don't ship evo_mail_notifier.png
Comment 5 Milan Crha 2010-05-19 11:12:27 UTC
Thanks, patch works good, it fixed some "compiler" warnings too.

Created commit b94a22e in evo master (2.31.2+)
Comment 6 Matthew Barnes 2010-07-25 10:28:20 UTC
Removed some localized versions of this image in master:

http://git.gnome.org/browse/evolution/commit/?id=68aa811ea45a0b68c3d675ec9ccdaaf5159daae5

And backported it all to gnome-2-30:

http://git.gnome.org/browse/evolution/commit/?h=gnome-2-30&id=1a375dd3d8a2594851a67a648952c6913fb49f74
Comment 7 Akhil Laddha 2010-09-30 06:46:36 UTC
*** Bug 607052 has been marked as a duplicate of this bug. ***