<!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, 2011-03-23 at 15:07 +0000, Jon TURNEY wrote:
<BLOCKQUOTE TYPE=CITE>
<PRE>
On 14/03/2011 18:23, Gaetan Nadon wrote:
&gt; On Mon, 2011-03-14 at 08:18 -0700, Dan Nicholson wrote:
&gt; 
&gt;&gt; I think to be correct you need -I$(top_srcdir) and -I$(top_builddir)
&gt;&gt; so you get both sessreg.h and config.h (included through sessreg.h).
&gt;&gt; The alternative is to leave filenames.sed.c in the top directory and
&gt;&gt; ensure it gets built before descending to the man directory.
&gt; 
&gt; 
&gt; Sounds good, there should be no -I.. ragardless.
&gt; 
&gt; I have posted a set of remaining patches to replace the cpp stuff with
&gt; plain robust man pages.
&gt; I had found many issues with the current man pages in terms of content.
&gt; These patches also fix the current build problem.

Attached is an updated patch using both -I$(top_srcdir) and -I$(top_builddir),
I'm not sure why I thought just -I$(top_srcdir) was sufficent.

If you aren't going forward with your other suggested patches to sessreg, can
I get a r-b: for this?
</PRE>
</BLOCKQUOTE>
<BR>
Sure.<BR>
<BR>
Reviewed-by: Gaetan Nadon &lt;<A HREF="mailto:memsize@videotron.ca">memsize@videotron.ca</A>&gt;
<BLOCKQUOTE TYPE=CITE>
<PRE>
_______________________________________________
<A HREF="mailto:xorg-devel@lists.x.org">xorg-devel@lists.x.org</A>: X.Org development
Archives: <A HREF="http://lists.x.org/archives/xorg-devel">http://lists.x.org/archives/xorg-devel</A>
Info: <A HREF="http://lists.x.org/mailman/listinfo/xorg-devel">http://lists.x.org/mailman/listinfo/xorg-devel</A>
</PRE>
</BLOCKQUOTE>
</BODY>
</HTML>