[PATCH] INSTALL doesn't belong in docdir

[Gaetan Nadon]

On Sat, 2010-01-02 at 14:28 -0500, Gaetan Nadon wrote:

> On Sat, 2010-01-02 at 11:12 +0000, Julien Cristau wrote: 
> 
> > 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.
> 
> 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. 
> 

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.




> > 
> > Cheers,
> > Julien
> 
> _______________________________________________
> xorg-devel mailing list
> xorg-devel at lists.x.org
> http://lists.x.org/mailman/listinfo/xorg-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.x.org/archives/xorg-devel/attachments/20100102/6c84efbb/attachment.html