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 401263 - Image files don't honour ALL_LINGUAS during build.
Image files don't honour ALL_LINGUAS during build.
Status: RESOLVED FIXED
Product: GIMP-manual
Classification: Other
Component: General
unspecified
Other All
: Normal enhancement
: ---
Assigned To: Róman Joost
Róman Joost
Depends on:
Blocks:
 
 
Reported: 2007-01-27 08:21 UTC by Peter
Modified: 2007-01-28 17:04 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Makefile.am patch. (2.05 KB, patch)
2007-01-27 08:24 UTC, Peter
none Details | Review

Description Peter 2007-01-27 08:21:18 UTC
Currently gimp-doc does not use ALL_LINGUAS during images installation
and thus installs all images even if ALL_LINGUAS was set only to en.
This increases building time if configured with --enable-convert as more
pictures require "Converting..." and also this increase disk usage of
installed documentation (whole installation occupies 37934036 bytes (42M) and if only images for en and ru installed 20808336 bytes (24M)).
Comment 1 Peter 2007-01-27 08:24:04 UTC
Created attachment 81317 [details] [review]
Makefile.am patch.

With the feedback gathered on the mailing list and in our bugzilla this patch:

 * Always install pictures in listed in IMAGE_DIRS (images for en)
 * process of image copying should abort on error

With this changes I'd like to point your attention at two things:
1. Now it's possible to create "en" directory for images which are required only for English language. Build process will not install this images if "en" it not in ALL_LINGUAS
2. If you ever create new base images subdirectory (in other words subdirectory which is *not* $lang subdirectory) it's necessary to list that directory in IMAGE_DIRS inside Makefile.am. This change I've documented in README (another patch in attachment)
Comment 2 Róman Joost 2007-01-27 09:00:52 UTC
Thanks Peter. The workaround of saving English images in it's seperate directory makes no sense to me. We use English as a fallback anyways, so including them is the way to go.

The only problem we now have is the point, that the 'find' command on OS X doesn't honour -wholename. Would it be possible to use a similar option?
Comment 3 Róman Joost 2007-01-27 09:06:10 UTC
--> CONFIRMED
Comment 4 Peter 2007-01-27 10:59:27 UTC
Roman: Sorry I was not clear about -wholename here, see mailing list for more details. Current implementation does not use -wholename. :)
Comment 5 Róman Joost 2007-01-28 12:44:50 UTC
Ow - you made the patch against Makefile.am revision 1583. Have to test it with this revision...
Comment 6 Róman Joost 2007-01-28 16:44:29 UTC
This patch seems to work now. I'll check it in.

-> assign
Comment 7 Róman Joost 2007-01-28 17:04:20 UTC
patch applied with revision 1694. Thanks Peter!

-> fixed