<!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.26.0">
</HEAD>
<BODY>
On Sat, 2010-01-02 at 11:06 +0000, Julien Cristau wrote:
<BLOCKQUOTE TYPE=CITE>
<PRE>
docdir stays in xorg-macros.pc so we don't break the existing
$(INSTALL_CMD).
---
Something like this should move INSTALL to a more appropriate location.
</PRE>
</BLOCKQUOTE>
I agree the new location is more appropriate.
<BLOCKQUOTE TYPE=CITE>
<PRE>

 Makefile.am       |    2 +-
 configure.ac      |    2 +-
 xorg-macros.m4.in |   10 +++++-----
 xorg-macros.pc.in |    5 +++--
 4 files changed, 10 insertions(+), 9 deletions(-)

diff --git a/Makefile.am b/Makefile.am
index 1e07014..7340beb 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -21,7 +21,7 @@
 
 aclocaldir = $(datadir)/aclocal
 aclocal_DATA = xorg-macros.m4
-dist_doc_DATA = INSTALL
+dist_pkgdata_DATA = INSTALL
 
 # Previous versions used to install xorgversion.m4, now integrated
 # into xorg-macros.m4.  Explicitly remove that old file in order not
diff --git a/configure.ac b/configure.ac
index 99c8996..ad244ba 100644
--- a/configure.ac
+++ b/configure.ac
@@ -23,7 +23,7 @@ dnl Process this file with autoconf to create configure.
 
 AC_PREREQ([2.60])
 AC_INIT([util-macros],
-        [1.4.1],
+        [1.5.0],
         [<A HREF="https://bugs.freedesktop.org/enter_bug.cgi?product=xorg">https://bugs.freedesktop.org/enter_bug.cgi?product=xorg</A>],
         [util-macros])
 AC_CONFIG_SRCDIR([Makefile.am])
diff --git a/xorg-macros.m4.in b/xorg-macros.m4.in
index 20d0c70..9d70ac6 100644
--- a/xorg-macros.m4.in
+++ b/xorg-macros.m4.in
@@ -504,17 +504,17 @@ XORG_MANPAGE_SECTIONS
 
 # XORG_INSTALL()
 # ----------------
-# Minimum version: 1.4.0
+# Minimum version: 1.5.0
 #
 # Defines the variable INSTALL_CMD as the command to copy
-# INSTALL from $prefix/share/doc/util-macros.
+# INSTALL from $prefix/share/util-macros.
 #
 AC_DEFUN([XORG_INSTALL], [
 AC_REQUIRE([PKG_PROG_PKG_CONFIG])
-macros_docdir=`$PKG_CONFIG --print-errors --variable=docdir xorg-macros`
-INSTALL_CMD=&quot;(cp -f &quot;$macros_docdir/INSTALL&quot; \$(top_srcdir)/.INSTALL.tmp &amp;&amp; \
+macros_datadir=`$PKG_CONFIG --print-errors --variable=pkgdata xorg-macros`
</PRE>
</BLOCKQUOTE>
-------------------------------------------------------------------------------------------------&gt; <FONT COLOR="#000000"><B>pkgdatadir</B></FONT><FONT COLOR="#000000"> perhaps?</FONT><FONT COLOR="#ff0000"></FONT>
<BLOCKQUOTE TYPE=CITE>
<PRE>
+INSTALL_CMD=&quot;(cp -f &quot;$macros_datadir/INSTALL&quot; \$(top_srcdir)/.INSTALL.tmp &amp;&amp; \
 mv \$(top_srcdir)/.INSTALL.tmp \$(top_srcdir)/INSTALL) \
 || (rm -f \$(top_srcdir)/.INSTALL.tmp; touch \$(top_srcdir)/INSTALL; \
-echo 'util-macros \&quot;docdir\&quot; from xorg-macros.pc not found: installing possibly empty INSTALL.' &gt;&amp;2)&quot;
+echo 'util-macros \&quot;pkgdatadir\&quot; from xorg-macros.pc not found: installing possibly empty INSTALL.' &gt;&amp;2)&quot;
 AC_SUBST([INSTALL_CMD])
 ]) # XORG_INSTALL
diff --git a/xorg-macros.pc.in b/xorg-macros.pc.in
index 15a1154..d3f7408 100644
--- a/xorg-macros.pc.in
+++ b/xorg-macros.pc.in
@@ -3,9 +3,10 @@ exec_prefix=@exec_prefix@
 includedir=@includedir@
 datarootdir=@datarootdir@
 datadir=@datadir@
-PACKAGE_TARNAME=@PACKAGE_TARNAME@
+PACKAGE=@PACKAGE@
 # Used by XORG_INSTALL
-docdir=@docdir@
+pkgdatadir=${datadir}/${PACKAGE}
+docdir=${pkgdatadir}
</PRE>
</BLOCKQUOTE>
&nbsp;&nbsp;&nbsp;&nbsp; docdir: a comment about backward compatibility would be useful for other people a couple of years down the road.
<BLOCKQUOTE TYPE=CITE>
<PRE>
 
 Name: X.Org Macros
 Description: A set of autoconf project macros for X.Org modules
</PRE>
</BLOCKQUOTE>
<BR>
I'll wait for your reply and give it a good test.<BR>
<BR>
Regards,<BR>
Gaetan<BR>
<BR>
</BODY>
</HTML>