<!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 Fri, 2009-11-27 at 09:26 -0800, Dan Nicholson wrote:
<BLOCKQUOTE TYPE=CITE>
<PRE>
On Thu, Nov 26, 2009 at 7:20 PM, Peter Hutterer
&lt;<A HREF="mailto:peter.hutterer@who-t.net">peter.hutterer@who-t.net</A>&gt; wrote:
&gt;&gt; From 1e50b971bf7273f958e11cbcf3dc07a6fab04335 Mon Sep 17 00:00:00 2001
&gt;&gt; From: Gaetan Nadon &lt;<A HREF="mailto:memsize@videotron.ca">memsize@videotron.ca</A>&gt;
&gt;&gt; Date: Sat, 21 Nov 2009 17:28:32 -0500
&gt;&gt; Subject: [PATCH] xorg-macros.pc.in: create a meta data file for util-macros
&gt;&gt;
&gt;&gt; This is motivated primarly by XORG_INSTALL who needs to locate
&gt;&gt; the docdir when invoked from any module that may not be installed
&gt;&gt; using same prefix (default or otherwise).
&gt;&gt;
&gt;&gt; There are other potential use for this file.
&gt;&gt; ---
&gt;&gt; &nbsp;Makefile.am &nbsp; &nbsp; &nbsp; | &nbsp; &nbsp;5 +++++
&gt;&gt; &nbsp;configure.ac &nbsp; &nbsp; &nbsp;| &nbsp; &nbsp;1 +
&gt;&gt; &nbsp;xorg-macros.m4.in | &nbsp; &nbsp;9 ++++++---
&gt;&gt; &nbsp;xorg-macros.pc.in | &nbsp; 14 ++++++++++++++
&gt;&gt; &nbsp;4 files changed, 26 insertions(+), 3 deletions(-)
&gt;&gt; &nbsp;create mode 100644 xorg-macros.pc.in
&gt;&gt;
&gt;&gt; diff --git a/Makefile.am b/Makefile.am
&gt;&gt; index 49c2d17..1e07014 100644
&gt;&gt; --- a/Makefile.am
&gt;&gt; +++ b/Makefile.am
&gt;&gt; @@ -29,6 +29,11 @@ dist_doc_DATA = INSTALL
&gt;&gt; &nbsp;install-data-hook:
&gt;&gt; &nbsp; &nbsp; &nbsp; rm -f $(DESTDIR)$(aclocaldir)/xorgversion.m4
&gt;&gt;
&gt;&gt; +pkgconfigdir = $(datadir)/pkgconfig
&gt;&gt; +pkgconfig_DATA = xorg-macros.pc
&gt;&gt; +
&gt;&gt; +EXTRA_DIST = xorg-macros.pc.in
&gt;&gt; +
&gt;&gt; &nbsp;.PHONY: ChangeLog
&gt;&gt;
&gt;&gt; &nbsp;ChangeLog:
&gt;&gt; diff --git a/configure.ac b/configure.ac
&gt;&gt; index 8b1d450..4daec6a 100644
&gt;&gt; --- a/configure.ac
&gt;&gt; +++ b/configure.ac
&gt;&gt; @@ -40,5 +40,6 @@ m4_include([xorgversion.m4])
&gt;&gt; &nbsp;XORG_RELEASE_VERSION
&gt;&gt; &nbsp;XORG_CHANGELOG
&gt;&gt;
&gt;&gt; +AC_CONFIG_FILES([xorg-macros.pc])
&gt;&gt; &nbsp;AC_CONFIG_FILES([Makefile xorg-macros.m4:xorg-macros.m4.in:xorgversion.m4])
&gt;&gt; &nbsp;AC_OUTPUT
&gt;&gt; diff --git a/xorg-macros.m4.in b/xorg-macros.m4.in
&gt;&gt; index 336490a..6f9635f 100644
&gt;&gt; --- a/xorg-macros.m4.in
&gt;&gt; +++ b/xorg-macros.m4.in
&gt;&gt; @@ -509,8 +509,11 @@ XORG_MANPAGE_SECTIONS
&gt;&gt; &nbsp;# INSTALL from $prefix/share/doc/util-macros.
&gt;&gt; &nbsp;#
&gt;&gt; &nbsp;AC_DEFUN([XORG_INSTALL], [
&gt;&gt; -INSTALL_CMD=&quot;if test -f &quot;$prefix/share/doc/util-macros/INSTALL&quot;; then \
&gt;&gt; -cp -f &quot;$prefix/share/doc/util-macros/INSTALL&quot; \$(top_srcdir); \
&gt;&gt; -else echo '$prefix/share/doc/util-macros/INSTALL cannot be found.' &gt;&amp;2; fi &quot;
&gt;&gt; +AC_REQUIRE([PKG_PROG_PKG_CONFIG])
&gt;&gt; +macros_docdir=$($PKG_CONFIG --print-errors --variable=docdir xorg-macros)

Please don't use $( ) for shell command substitution. ` ` is more portable.

</PRE>
</BLOCKQUOTE>
Thanks! That's an area I need to learn more about. I picked this line from other .ac files. I'll make it a todo when I review those to make the correction. I did replace &quot;pkg-config&quot; with the libtool env var in the hope it would be more portable and/or safer.<BR>
<BR>
sdkdir=$(pkg-config --variable=sdkdir xorg-server)<BR>
<BR>
As a matter of timing, I had just pushed the patch before I got your comment. I will do it soon.<BR>
<BR>
<BR>
<BLOCKQUOTE TYPE=CITE>
<PRE>
With that change,

Reviewed-by: Dan Nicholson &lt;<A HREF="mailto:dbn.lists@gmail.com">dbn.lists@gmail.com</A>&gt;

--
Dan
</PRE>
</BLOCKQUOTE>
</BODY>
</HTML>