[Xcb] Re: [Xcb-commit] configure.ac src

[Vincent Torri]

Hey,

On Mon, 27 Nov 2006, Jamey Sharp wrote:

> On Thu, Nov 23, 2006 at 08:05:54AM +0100, Vincent Torri wrote:
>> Then, could you please add at the end of the output of configure the value
>> of the used flags, in order for the user to know which ones will be used
>> for the compilation ?
>>
>> That would allow someone to see if he forgot to set some flags
>
> This is a good idea, but I don't think Josh or I can get to it soon.
> Please feel free to add some summary output. I think the libX11 or cairo
> summaries are decent models. You might consider selecting a few other
> interesting values from the configure script, like whether XCB is being
> built to support XDM-AUTHORIZATION-1 using libXdmcp.

I've attached a patch. Does it look good enough ?

Vincent
-------------- next part --------------
diff --git a/configure.ac b/configure.ac
index b400f03..08706d2 100644
--- a/configure.ac
+++ b/configure.ac
@@ -34,11 +34,13 @@ PKG_CHECK_MODULES(XCBPROTO, xcb-proto >=
 NEEDED="xau pthread-stubs"
 PKG_CHECK_MODULES(NEEDED, $NEEDED)
 
+have_xdmcp="no"
 PKG_CHECK_MODULES(XDMCP, xdmcp,
 	AC_CHECK_LIB(Xdmcp, XdmcpWrap,
 		[
 		AC_DEFINE(HASXDMAUTH,1,[Has Wraphelp.c needed for XDM AUTH protocols])
 		NEEDED="$NEEDED xdmcp"
+		have_xdmcp="yes"
 		],
 		[
 		XDMCP_CFLAGS=
@@ -87,3 +89,25 @@ AC_CONFIG_FILES([xcb.pc xcb-xlib.pc xcb-
 AC_CONFIG_FILES([doc/xcb.doxygen])
 
 AC_OUTPUT
+
+dnl Configuration output
+build_tests="no"
+if test "x${HTML_CHECK_RESULT}" = "xtrue" ; then
+    build_tests="yes"
+fi
+
+echo ""
+echo "  Package: ${PACKAGE_NAME} ${PACKAGE_VERSION}"
+echo ""
+echo "  Configuration"
+echo "    XDM support.........: ${have_xdmcp}"
+echo "    Build unit tests....: ${build_tests}"
+echo ""
+echo "  Used CFLAGS:"
+echo "    CPPFLAGS............: ${CPPFLAGS}"
+echo "    CFLAGS..............: ${CFLAGS}"
+echo "    Warning CFLAGS......: ${CWARNFLAGS}"
+echo ""
+echo "  Installation:"
+echo "    Prefix..............: ${prefix}"
+echo ""