<!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 14:28 -0500, Gaetan Nadon wrote: <BLOCKQUOTE TYPE=CITE> On Sat, 2010-01-02 at 11:12 +0000, Julien Cristau wrote: <BLOCKQUOTE TYPE=CITE> <PRE> On Sat, Jan 2, 2010 at 11:06:05 +0000, Julien Cristau wrote: > -macros_docdir=`$PKG_CONFIG --print-errors --variable=docdir xorg-macros` > -INSTALL_CMD="(cp -f "$macros_docdir/INSTALL" \$(top_srcdir)/.INSTALL.tmp && \ > +macros_datadir=`$PKG_CONFIG --print-errors --variable=pkgdata xorg-macros` bah.  make that --variable=pkgdatadir. </PRE> </BLOCKQUOTE> Ok. I had not read this e-mail yet. I was informed a while back that bumping the module version has to be done in a separate patch for administrative reasons. </BLOCKQUOTE> I did some test scenario regarding backward compatibility. It turns out setting docdir=pkgdatadir does not help. The value of docdir in the module's copy of INSTALL_CMD is set at configuration time. If the module was configured with macros @ v1.4, then the INSTALL_CMD will copy from share/doc every time the makefile target is invoked. INSTALL_CMD = (cp -f /home/nadon/xorg/src/share/doc/util-macros/INSTALL ... When the file is missing, a zero-byte file is created and the target has a successful completion code. This is the behaviour by design for both INSTALL and ChangeLog. When creating tarballs on x.org site, the system is in a consistent state, so these are always created. I would leave the value of docdir as it is, given that it is correct. It will simply be unused. <BLOCKQUOTE TYPE=CITE> <BLOCKQUOTE TYPE=CITE> <PRE> Cheers, Julien </PRE> </BLOCKQUOTE> <PRE> _______________________________________________ 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>