Re: gettext-0.12 is released

[Pavel Roskin <proski gnu org>]

On Tue, 20 May 2003, Bruno Haible wrote:

> Available at
>     ftp://ftp.gnu.org/pub/gnu/gettext/gettext-0.12.tar.gz

I'm dropping most cc:, but please feel free to forward my e-mail to the
list for gettext developers if there is one.

It looks like version 0.12 has problems with builds outside the source
tree, at least when libtool is not used.  Trying to use it with GNU
Midnight Commander causes compile error in "make distcheck".  Manual
out-of-tree build also fails:

make[2]: Entering directory `/usr/local/src/mc.v0/mc-4.6.0a/build/intl'
make[2]: *** No rule to make target `bindtextdom.c', needed by
`bindtextdom.o'.  Stop.

Indeed, intl/Makefile in the build directory has explicit rules for *.lo
with $(srcdir) and suffix rule for *.o, which doesn't work because VPATH
is not set.

Either the explicit rules should be written for *.o files, or VPATH should
be restored.  It was in intl/Makefile in gettext-0.11.5.

Please note that the line beginning with VPATH in every Makefile is
removed by config.status for the in-tree builds.  This is an exception.
Normally config.status only substitutes variables surrounded by @-signs.
So having VPATH shouldn't affect the in-tree builds.  However, out-of-tree
builds need it.

VPATH was removed in revision 1.2 of gettext-runtime/intl/Makefile.in:

revision 1.2
date: 2003/02/14 20:25:01;  author: haible;  state: Exp;  lines: +148 -61
Adapt to new structure of gettext package.

The patch that restores VPATH is attached.

I would have found the problem is a prerelease if I could have used
AM_GNU_GETTEXT_VERSION(0.12-preX).  It didn't work, I reported that, you
replied that it would take too much work to make "complete" prereleases.
I didn't have time to install the prerelease manually and run it through
"make distcheck".

-- 
Regards,
Pavel Roskin
--- gettext-runtime/intl/Makefile.in
+++ gettext-runtime/intl/Makefile.in
@@ -24,6 +24,7 @@
 srcdir = @srcdir@
 top_srcdir = @top_srcdir@
 top_builddir = ..
+VPATH = @srcdir@
 
 prefix = @prefix@
 exec_prefix = @exec_prefix@