<!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 Wed, 2009-11-25 at 12:55 +1000, Peter Hutterer wrote: 
<BLOCKQUOTE TYPE=CITE>
<PRE>
On Sun, Nov 22, 2009 at 03:59:44PM -0500, Gaetan Nadon wrote:
&gt; 
&gt; Thanks to Daniel Stone for providing the idea of using a .pc file. This
&gt; provides a robust implementation to copy the INSTALL file from a single
&gt; location to all xorg modules.  It can be updated every release if need
&gt; be with a single check-in.
&gt; 
&gt; The time honoured behaviours of ChangeLog are replicated.  When the file
&gt; cannot be copied, the existing one, if any, is not overwritten. If none
&gt; existed, an empty file is created so the tarball is created anyway.
&gt; Similar to ChangeLog, if the file is deleted from a tarball and
&gt; util-macros is not available, then it cannot be created. In that case,
&gt; one would have to obtain it from the original tarball or from git.
&gt; 
&gt; Note that xorg-macros.pc is installed in $prefix/share/pkgconfig because
&gt; it is architecture independent. I recalled a discussion about libxtrans
&gt; that could not cross compile.
&gt; 
&gt; I will release an additional patch for each module  to add both
&gt; ChangeLog and INSTALL on the MAINTAINERCLEANFILES list.
&gt; 
&gt; There has been questions as to what a module should do when additional
&gt; information is required, now that the INSTALL is standard. Here is an
&gt; excerpt fromthe INSTALL file:
&gt; 
&gt; Basic Installation
&gt; ==================
&gt; 
&gt;    Briefly, the shell commands `./configure; make; make install' should
&gt; configure, build, and install this package.  The following
&gt; more-detailed instructions are generic; see the `README' file for
&gt; instructions specific to this package.
&gt; 
&gt; 
&gt; Thank you very much for all those who helped with this issue.
&gt; 
&gt; The content of the XORG_INSTALL macro:
&gt; 
&gt; AC_REQUIRE([PKG_PROG_PKG_CONFIG])
&gt; macros_docdir=$($PKG_CONFIG --print-errors --variable=docdir
&gt; xorg-macros)
&gt; INSTALL_CMD=&quot;(cp -f &quot;$macros_docdir/INSTALL&quot; \$(top_srcdir)/.INSTALL.tmp
&gt; &amp;&amp; \
&gt; mv \$(top_srcdir)/.INSTALL.tmp \$(top_srcdir)/INSTALL) \
&gt; || (rm -f \$(top_srcdir)/.INSTALL.tmp; touch \$(top_srcdir)/INSTALL; \
&gt; echo 'util-macros \&quot;docdir\&quot; from xorg-macros.pc not found: installing

was this supposed to be in this patch?
it'd make sense to include it to this patch instead of a separate one since
the reason for the .pc file is this very macro.
</PRE>
</BLOCKQUOTE>
<BR>
There is only one patch. It was provided as an attachment. In the text, I provided an excerpt. I realize now it's not a good idea when the attachment is viewed in-line. I have re-attached the patch to this e-mail. 
<BLOCKQUOTE TYPE=CITE>
<PRE>


&gt; From 2c10c1b272a9388e7ef3f8a6e1f723acea3be5af Mon Sep 17 00:00:00 2001
&gt; From: Gaetan Nadon &lt;<A HREF="mailto:memsize@videotron.ca">memsize@videotron.ca</A>&gt;
&gt; Date: Sat, 21 Nov 2009 17:28:32 -0500
&gt; Subject: [PATCH] xorg-macros.pc.in: create a meta data file for util-macros
&gt; 
&gt; This is motivated primarly by XORG_INSTALL who needs to locate
&gt; the docdir when invoked from any module that may not be installed
&gt; using same prefix (default or otherwise).
&gt; 
&gt; There are other potential use for this file.
&gt; ---
&gt;  Makefile.am       |    5 +++++
&gt;  configure.ac      |    1 +
&gt;  xorg-macros.pc.in |   14 ++++++++++++++
&gt;  3 files changed, 20 insertions(+), 0 deletions(-)
&gt;  create mode 100644 xorg-macros.pc.in
&gt; 
&gt; diff --git a/Makefile.am b/Makefile.am
&gt; index 49c2d17..1e07014 100644
&gt; --- a/Makefile.am
&gt; +++ b/Makefile.am
&gt; @@ -29,6 +29,11 @@ dist_doc_DATA = INSTALL
&gt;  install-data-hook:
&gt;          rm -f $(DESTDIR)$(aclocaldir)/xorgversion.m4
&gt;  
&gt; +pkgconfigdir = $(datadir)/pkgconfig
&gt; +pkgconfig_DATA = xorg-macros.pc
&gt; +
&gt; +EXTRA_DIST = xorg-macros.pc.in
&gt; +
&gt;  .PHONY: ChangeLog
&gt;  
&gt;  ChangeLog:
&gt; diff --git a/configure.ac b/configure.ac
&gt; index 8b1d450..4daec6a 100644
&gt; --- a/configure.ac
&gt; +++ b/configure.ac
&gt; @@ -40,5 +40,6 @@ m4_include([xorgversion.m4])
&gt;  XORG_RELEASE_VERSION
&gt;  XORG_CHANGELOG
&gt;  
&gt; +AC_CONFIG_FILES([xorg-macros.pc])
&gt;  AC_CONFIG_FILES([Makefile xorg-macros.m4:xorg-macros.m4.in:xorgversion.m4])

any reason that can't be in a single command, like AC_OUTPUT in most other
modules?  i.e. 

AC_CONFIG_FILES([xorg-macros.pc
                 Makefile
                 etc])

</PRE>
</BLOCKQUOTE>
For small modules, it's pretty much just a style issue. AC_CONFIG_FILES is an instantiating macro, so it can have commands.<BR>
<BR>
Macro: <B>AC_CONFIG_FILES</B> (<TT>file</TT><TT><FONT SIZE="2">...</FONT></TT><TT>, </TT>[<TT>cmds</TT>]<TT>, </TT>[<TT>init-cmds</TT>])<BR>
<BR>
For large config file like libxcb, they have grouped files by topic, makefile together, .pc together and finally doxygen. If commands are needed, then they must have separate statement. I can write a single statement at the next patch opportunity. Thanks for the advanced warning, it will be something to ask when the opportunity arise.<BR>
<BR>
Thanks<BR>
<BR>
<BLOCKQUOTE TYPE=CITE>
<PRE>
Cheers,
  Peter
_______________________________________________
xorg-devel mailing list
<A HREF="mailto:xorg-devel@lists.x.org">xorg-devel@lists.x.org</A>
<A HREF="http://lists.x.org/mailman/listinfo/xorg-devel">http://lists.x.org/mailman/listinfo/xorg-devel</A>
</PRE>
</BLOCKQUOTE>
</BODY>
</HTML>