<!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, 2010-08-13 at 12:49 +0200, Arnaud Fontaine wrote:<BR>
<BLOCKQUOTE TYPE=CITE>
    <TT><FONT COLOR="#1a1a1a">I also&nbsp; had a look&nbsp; at m4 on&nbsp; various OS (including Solaris,&nbsp; Darwin and</FONT></TT><BR>
    <TT><FONT COLOR="#1a1a1a">FreeBSD), and apparently only m4&nbsp; on Solaris does not support -I option,</FONT></TT><BR>
    <TT><FONT COLOR="#1a1a1a">which&nbsp; is not&nbsp; surprising because&nbsp; POSIX only&nbsp; requires support&nbsp; for the</FONT></TT><BR>
    <TT><FONT COLOR="#1a1a1a">command line options -s, -D, and -U. Thus, I think the name of the macro</FONT></TT><BR>
    <TT><FONT COLOR="#1a1a1a">is a bit&nbsp; confusing, so I renamed it&nbsp; to XORG_M4_WITH_INCLUDE_PATH. Hope</FONT></TT><BR>
    <TT><FONT COLOR="#1a1a1a">that's ok.</FONT></TT><BR>
    <BR>
</BLOCKQUOTE>
I tried the patch, it looks perfect. I like the commit message and comment.<BR>
This is the logic I attempted to code in my patch.<BR>
<BR>
Given your research above, suppose we were to use AC_PATH_RPOGS with gm4 first and then m4.<BR>
I think it would cover all (real life) scenarios correctly.&nbsp; If that does not work, you can always fix my patch.<BR>
<BR>
Gaetan<BR>
<BR>
</BODY>
</HTML>