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 749866 - broken dist tarball creates directories in /usr/locale
broken dist tarball creates directories in /usr/locale
Status: RESOLVED FIXED
Product: adwaita-icon-theme
Classification: Core
Component: general
3.16.x
Other Linux
: Normal major
: ---
Assigned To: adwaita-icon-theme maintainer(s)
adwaita-icon-theme maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2015-05-25 20:21 UTC by Michael Biebl
Modified: 2015-06-12 20:28 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Fix intltool locale directory location (758 bytes, patch)
2015-06-12 13:02 UTC, Jakub Steiner
committed Details | Review

Description Michael Biebl 2015-05-25 20:21:16 UTC
Version: 3.16.2.1

The tarball for 3.16.2.1 [1] has a broken build systemd which results in
the package creating directories in /usr/locale [2] due to DATADIRNAME being undefined.
Afaics, the problem seems to be a broken m4/intltool.m4, which doesn't
properly define DATADIRNAME.

Would we great if you can spin a new tarball.
I don't know, on which platform the tarball was created and which intltool version was installed.
Do you have an idea, how we can avoid that this issue happens again?



[1] https://download.gnome.org/sources/adwaita-icon-theme/3.16/adwaita-icon-theme-3.16.2.tar.xz
[2] https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=786723
Comment 1 Sebastien Bacher 2015-06-11 17:16:23 UTC
it's because DATADIRNAME has been removed from intltool
https://bazaar.launchpad.net/~intltool/intltool/trunk/revision/742
Comment 2 Michael Biebl 2015-06-11 17:22:11 UTC
(In reply to Sebastien Bacher from comment #1)
> it's because DATADIRNAME has been removed from intltool
> https://bazaar.launchpad.net/~intltool/intltool/trunk/revision/742

has anyone checked how many packages are broken by that?

This intltool change seems to at least break
adwaita-icon-theme and gnome-icon-theme
Comment 3 Jakub Steiner 2015-06-12 13:02:10 UTC
Created attachment 305143 [details] [review]
Fix intltool locale directory location

Recent intltool versions no longer set DATADIRNAME.
Comment 4 Jakub Steiner 2015-06-12 13:02:40 UTC
Attachment 305143 [details] pushed as f07e611 - Fix intltool locale directory location
Comment 5 Michael Biebl 2015-06-12 20:28:43 UTC
I grepped through some of the git repos I have checked out:

avahi/configure.ac:avahilocaledir='${prefix}/${DATADIRNAME}/locale'
brasero/configure.ac:  AC_DEFINE_UNQUOTED(PACKAGE_LOCALE_DIR, "${ac_default_prefix}/${DATADIRNAME}/locale", [Location of package locale files])
brasero/configure.ac:  AC_DEFINE_UNQUOTED(PACKAGE_LOCALE_DIR, "${prefix}/${DATADIRNAME}/locale", [Location of package locale files])
file-roller/configure.ac:  AC_DEFINE_UNQUOTED(LOCALEDIR, "${ac_default_prefix}/${DATADIRNAME}/locale", [Locale dir])
file-roller/configure.ac:  AC_DEFINE_UNQUOTED(LOCALEDIR, "${prefix}/${DATADIRNAME}/locale", [Locale dir])
gnome-applets/configure.ac:gnomelocaledir='${prefix}/${DATADIRNAME}/locale'
libwnck/configure.ac:# AM_GLIB_GNU_GETTEXT above substs $DATADIRNAME
libwnck/configure.ac:wncklocaledir='${prefix}/${DATADIRNAME}/locale'


That list is by no means complete.

I guess, they all should be updated to use autoconf 2.60 and @localedir@
https://www.gnu.org/software/gettext/manual/html_node/src_002fMakefile.html