Re: [Adonthell-devel] Crash in mapedit

[Chris Frey]

On Mon, Aug 01, 2011 at 06:06:14PM +0200, Kai Sterker wrote:
> On Mon, Aug 1, 2011 at 9:37 AM, Josef V??t?? <address@hidden> wrote:
> 
> > I tried to reproduce the error with config.rpath. Apparently it is
> > not copied by autopoint in my case, but gettextize instead
> > (autogen.sh:11). autopoint is not needed at all.
> >
> > The configure script does not care at all about the content of
> > config.rpath, therefore (just for the sake of compiling a working
> > program) gettextize is non-essential, 'touch' is enough. And, as
> > stated by Chris, empty config.rpath file causes no problems, as it
> > can be safely replaced by gettextize if installed.
> 
> You're right. I've added a (not quite) empty placeholder that will
> hopefully fix the issue if gettextize is not available.

I didn't mean to imply that a 0 byte file is the best solution. :-)

For me, autopoint does copy a version of config.rpath, but you need
to add the following to configure.in:

------------------

diff --git a/configure.in b/configure.in
index b5bba26..9f75ece 100644
--- a/configure.in
+++ b/configure.in
@@ -7,6 +7,9 @@ AC_INIT
 AC_CONFIG_SRCDIR([README])
 AC_CONFIG_AUX_DIR(.)
 
+AM_GNU_GETTEXT([external])
+AM_GNU_GETTEXT_VERSION([0.17])
+
 dnl Keep the correct libtool macros in-tree
 AC_CONFIG_MACRO_DIR([m4])
 

------------------
 
The AM_GNU_GETTEXT([external]) tells gettext to use an external libintl
if it can, I think.  They recommend [external] for packages that have
no intl/ directory of their own.

The AM_GNU_GETTEXT_VERSION([0.17]) tells autopoint which version of
gettext you're using, which lets it copy over the files needed for that
version.  Using version 0.17 has been pretty stable for me, with Barry.

Then you can update autoget.sh:

------------------

diff --git a/autogen.sh b/autogen.sh
index 0372fad..ddb355a 100755
--- a/autogen.sh
+++ b/autogen.sh
@@ -7,6 +7,7 @@ rm -f `find . -name 'Makefile.in'`
 rm -f `find . -name 'Makefile'`
 
 # Regenerate everything
+autopoint || echo "No autopoint, skipping..."
 aclocal -I .
 libtoolize --force --copy 
 autoheader

------------------

Autopoint runs pretty quick, so it shouldn't be too much of an imposition
to add.

Hope that helps,
- Chris