Bug#373643: Unsupported or non-substituted DATADIRNAME variable in Makefile.in.in template

[Daniel Leidert]

Am Donnerstag, den 15.06.2006, 09:46 +0200 schrieb Loïc Minier:
>         Hi,
> 
> On Wed, Jun 14, 2006, Daniel Leidert wrote:
> > No. It was a run from a clean CVS-copy running autogen.sh, which runs
> > intltoolize (the CVS-copy itself does not contain this file, so it is
> > freshly created from the file /usr/share/intltool/Makefile.in.in). I use
> > automake 1.9 and the latest autoconf package.
> 
>  Ok, could you try calling GLIB_WITH_NLS in configure.ac before
>  intltool?  Could you provide the necessary files to reproduce your
>  problem?  (perhaps the upstream CVS location if it's public)

I don't need the GLIB macros. I just ran AM_PO_SUBDIRS (or in the past,
I ran AM_GNU_GETTEXT, which is a bit overkill - but in bug #365727 I was
now told to use AM_PO_SUBDIRS for my purpose - atm I removed them
completely, because IT_PROG_INTLTOOL seems enough - except of the
DATADIRNAME variable issue). The necessity of the GLIB_* macros would be
some kind of overkill and a crap dependency, because the project uses
intltool for XML file internationalization and doesn't need any
libglib2.0-dev dependency. The code is available at
http://sourceforge.net/projects/chemical-mime/ CVS (the upload was done
some minutes ago, so anonymous CVS may lack the latest changes, I put
some archives to http://debian.wgdd.de/temp/
(chemical-mime-data-0.1.94.tar.{gz,bz2}).

http://chemical-mime.cvs.sourceforge.net/chemical-mime/chemical-mime-data/

run ./autogen && ./configure disable-update-database

then run make and try an installation with DESTDIR=whateveryouwant to
reproduce the issue. DATADIRNAME has no value.

> >                                               It is also a bit
> > surprising to me, that $itlocaledir does not make use of $datadir.
> > Instead it uses this (IMHO) crappy (sorry) expression
> > '$(prefix)/$(DATADIRNAME)/locale' Is there a special reason for this?
> 
>  One thing at a time.  :)

Well, the expression here uses the problematic DATADIRNAME variable,
which is not supported/not substituted. So I think, this is some related
information. Changing $itlocaledir to '$itlocaledir=$datadir/locale'
changes the problematic code to a working one.

>  (You can file another bug report on that particular problem.)
> 
> > Further the --localedir options was introduced recently. IMHO the used
> > expression will ignore this option. So IMHO, a check for configure.ac
> > should be added to test on $localedir and the current solution (without
> > the bug) should be the fallback (IMHO this is a suggestion for
> > upstream).
> 
>  I don't understand this last part.

The latest autoconf package adds support for a --localedir configure
option. So it is possible to define the location for the locales as
$localedir. This adds the possibility to add a check to configure.ac,
which tests, if $localedir is empty. If it is, the fallback could be
localedir=$datadir/locale or intltools (crappy) expression: localedir=
$(prefix)/$(DATADIRNAME)/locale. But this is really not directly related
to the issue I reported and something, I should tell upstream :)

Regards, Daniel