<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 TRANSITIONAL//EN">
<HTML>
<HEAD>
  <META HTTP-EQUIV="Content-Type" CONTENT="text/html; CHARSET=UTF-8">
  <META NAME="GENERATOR" CONTENT="GtkHTML/3.32.2">
</HEAD>
<BODY>
On Fri, 2011-09-09 at 07:43 +0000, Egbert Eich wrote:
<BLOCKQUOTE TYPE=CITE>
<PRE>
On Thu, Sep 08, 2011 at 06:03:18PM -0400, Gaetan Nadon wrote:
&gt; On Thu, 2011-09-08 at 16:44 +0200, Egbert Eich wrote:
&gt; 
&gt; Which expands to:
&gt; 
&gt;         datarootdir=${prefix}/share
&gt;         datadir=${datarootdir}
&gt;         errordbdir=${datarootdir}/X11

So the 'datadir=' line isn't needed unless we do

ERRORDBDIR='${datadir}/X11'
</PRE>
</BLOCKQUOTE>
Which is what you have coded in configure.ac. I recall you changed it from datarootdir. I was somewhat surprised to see it changed to datarootdir in the Makefile:<BR>
<BLOCKQUOTE>
    datadir = ${datarootdir}<BR>
    datarootdir = ${prefix}/share<BR>
    ...<BR>
    ERRORDBDIR = ${datarootdir}/X11<BR>
</BLOCKQUOTE>
<BR>
There is some history being all this. The datarootdir variable has not always existed and datadir had to be kept backward compatible. The semantic of datarootdir is the most appropriate for errordb location. It is the &quot;root directory of architecture independent data&quot;. If I set a value for datarootdir, all subdirectories (doc, locales, man pages, and errordbdir) are relocated as well. The datadir= line is not needed in the pc file. The makefile did the &quot;migration&quot; from datadir to datarootdir.<BR>
<BR>
<BLOCKQUOTE>
<PRE>
&quot;In Autoconf 2.60, the set of directory variables has changed, and the defaults of some variables have been adjusted 
(see <A HREF="http://www.gnu.org/software/autoconf/manual/autoconf.html#Installation-Directory-Variables">Installation Directory Variables</A>) to changes in the GNU Coding Standards. 
Notably, datadir, infodir, and mandir are now expressed in terms of datarootdir&quot;
</PRE>
</BLOCKQUOTE>
<BR>
<BLOCKQUOTE TYPE=CITE>
<PRE>

in configure.ac

&gt; 
&gt; You may also leave it as is and I can clean it later with appdefaultdir


While we are at it I can just as well fix my patch ;)
Will resend.

Cheers,
        Egbert.
</PRE>
</BLOCKQUOTE>
<BR>
</BODY>
</HTML>