<!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 Tue, 2010-08-24 at 22:46 +0200, Arnaud Fontaine wrote:<BR>
<BLOCKQUOTE TYPE=CITE>
    <TT><FONT COLOR="#1a1a1a">Would&nbsp; AC_PATH_PROGS_FEATURE_CHECK&nbsp; be&nbsp; useful&nbsp; for&nbsp; another&nbsp; macros&nbsp; of</FONT></TT><BR>
    <TT><FONT COLOR="#1a1a1a">util-macros? </FONT></TT><BR>
</BLOCKQUOTE>
Yes, we can find existing xorg packages where the use of this macro would reduce coding<BR>
(as it does for us in this case). Not having it, it's just a bit more code.<BR>
<BLOCKQUOTE TYPE=CITE>
    <TT><FONT COLOR="#1a1a1a">If&nbsp; not, I&nbsp; agree with&nbsp; you about adding&nbsp; the macro&nbsp; to xcb</FONT></TT><BR>
    <TT><FONT COLOR="#1a1a1a">(through&nbsp; the m4-common&nbsp; repository already&nbsp; used for&nbsp; macros&nbsp; common to</FONT></TT><BR>
    <TT><FONT COLOR="#1a1a1a">xcb-util&nbsp;&nbsp;&nbsp; libraries)&nbsp;&nbsp;&nbsp; rather&nbsp;&nbsp;&nbsp;&nbsp; than&nbsp;&nbsp;&nbsp; util-macros.&nbsp;&nbsp;&nbsp;&nbsp; In&nbsp;&nbsp;&nbsp; case</FONT></TT><BR>
    <TT><FONT COLOR="#1a1a1a">AC_PATH_PROGS_FEATURE_CHECK&nbsp; would&nbsp;&nbsp; be&nbsp; useful&nbsp; for&nbsp;&nbsp; other&nbsp; macros&nbsp; in</FONT></TT><BR>
    <TT><FONT COLOR="#1a1a1a">util-macros, why not just have something like:</FONT></TT><BR>
    <BR>
    <TT><FONT COLOR="#1a1a1a">m4_ifndef([AC_PATH_PROGS_FEATURE_CHECK], [</FONT></TT><BR>
    <TT><FONT COLOR="#1a1a1a">AC_DEFUN([AC_PATH_PROGS_FEATURE_CHECK], [COPY_OF_MACROS_FROM_AUTOCONF])</FONT></TT><BR>
    <TT><FONT COLOR="#1a1a1a">])</FONT></TT><BR>
    <TT><FONT COLOR="#1a1a1a">Then, as soon as Autoconf 2.62&nbsp; becomes the minimum version, we can just</FONT></TT><BR>
    <TT><FONT COLOR="#1a1a1a">get rid of that easily. What do you think?</FONT></TT><BR>
    <BR>
</BLOCKQUOTE>
I had a peek at the macro. It calls another macro which the author claims it's too ugly to make public.<BR>
I'd rather have a bit of repetitive&nbsp; code than a piece no one can understand. Otherwise I would do it, I have seen done in a few places.<BR>
<BR>
Bottom line I don't think it's worth the trouble. There isn't enough demand for it and the alternative is acceptable. All it really does is loop through the input and test each case. My m4 skills are limited, but this is something that can be done in an elegant way. <BR>
<BR>
Gaetan
</BODY>
</HTML>