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 132858 - glib-gettextize doesn't work with automake-1.8
glib-gettextize doesn't work with automake-1.8
Status: RESOLVED FIXED
Product: glib
Classification: Platform
Component: general
2.3.x
Other Linux
: High normal
: ---
Assigned To: gtkdev
gtkdev
: 138538 140176 141463 145308 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2004-01-29 15:05 UTC by Sven Neumann
Modified: 2011-02-18 16:07 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
glib-mkinstalldirs.patch (2.07 KB, patch)
2004-03-22 17:48 UTC, Stanislav Brabec
none Details | Review
glib-gettextize.patch (2.49 KB, patch)
2004-05-06 04:46 UTC, James Henstridge
none Details | Review

Description Sven Neumann 2004-01-29 15:05:02 UTC
The Makefile.in.in that is installed by glib-gettextize refers to
$(top_srcdir)/mkinstalldirs but this file is not installed by automake-1.8.
This causes 'make install' to break in the po directory for any projects
that use glib-gettextize and are built using automake-1.8.
Comment 1 Sven Neumann 2004-01-29 15:37:19 UTC
The best solution is probably to use $(mkinstalldirs) as defined by
automake instead of using $(top_srcdir)/@MKINSTALLDIRS@ as it is done
now. Even automake-1.8 defines $(mkinstalldirs) so this should fix it
for all versions.
Comment 2 Sven Neumann 2004-01-29 15:43:16 UTC
Sorry, but of course this can't work since Makefile.in.in isn't under
control of automake. Perhaps that's the basic problem here...
Comment 3 Sven Neumann 2004-01-29 16:28:29 UTC
FYI:

automake-1.8 defines $(mkinstalldirs) to @mkdir_p@ which is
substituted to "mkdir -p -- ." by configure. Perhaps that's a possible
solution.
Comment 4 Stanislav Brabec 2004-03-22 17:48:31 UTC
Created attachment 25890 [details] [review]
glib-mkinstalldirs.patch
Comment 5 Stanislav Brabec 2004-03-22 17:52:22 UTC
Attached patch works for me with automake 1.8, glib 2.2.3 and make
distcheck in test package.
Comment 6 Nickolay V. Shmyrev 2004-04-30 16:26:09 UTC
Probably, better solution will be to install ./mkinstalldirs in glib-gettextize
(gettextize do it). That will help to avoid problems with other files, that use
mkinstalldirs, like omf.make and xmldocs.make
Comment 7 Luis Villa 2004-04-30 16:33:15 UTC
*** Bug 140176 has been marked as a duplicate of this bug. ***
Comment 8 James Henstridge 2004-05-05 10:17:27 UTC
*** Bug 138538 has been marked as a duplicate of this bug. ***
Comment 9 James Henstridge 2004-05-05 10:31:36 UTC
The attached patch relies on substitutions created when you call Automake-1.8's
AM_INIT_AUTOMAKE (@mkdir_p@ comes from AM_PROG_MKDIR_P which is called by
AM_INIT_AUTOMAKE).

Given that this problem was caused by glib-gettextize relying on something the
Automake maintainers considered a side effect rather than a published API (ie.
"automake --add-missing" installs mkinstalldirs), I wouldn't check this in.

Also, automake < 1.8 doesn't define @mkdir_p@, so the patch will break
glib-gettextize's Makefile.in.in when used with older Automakes (such as the one
glib and GTK are using ...).

The two possible solutions are:

1. Get glib-gettextize to install mkinstalldirs.  This is what gettext's
   gettextize does.

2. swipe the macro that defines @mkdir_p@ and integrate it into
   AM_GLIB_GNU_GETTEXT (using different subst name, so as to not cause a clash)
   then adjust Makefile.in.in similar to how Stanislav did in his patch.
   The latest version of automake's macro is at:

http://savannah.gnu.org/cgi-bin/viewcvs/*checkout*/automake/automake/m4/mkdirp.m4
Comment 10 James Henstridge 2004-05-06 04:46:43 UTC
Created attachment 27413 [details] [review]
glib-gettextize.patch

A minimal patch that gets glib-gettextize to install mkinstalldirs.  Adapted
from upstream gettextize.
Comment 11 Damon Chaplin 2004-07-05 17:14:37 UTC
*** Bug 145308 has been marked as a duplicate of this bug. ***
Comment 12 Damon Chaplin 2004-07-12 10:43:35 UTC
*** Bug 141463 has been marked as a duplicate of this bug. ***