<html> <head> <meta content="text/html; charset=ISO-8859-1" http-equiv="Content-Type"> </head> <body bgcolor="#FFFFFF" text="#000000"> <div class="moz-cite-prefix">On 14-03-08 09:03 AM, Hans de Goede wrote: </div> <blockquote cite="mid:531B233C.7000102@redhat.com" type="cite">So I've just been looking into using AC_USE_SYSTEM_EXTENSIONS but that seem pretty useless, it only adds the relevant defines to confdefs.h, not into cflags, and in libxtrans we want to add them to the .pc file, so we need them in a variable. I'm open to different suggestions, but I think my initial patch might be the best way to fix this. </blockquote> Check the pattern in other modules such as libX11. The source code includes the content of the generated config.h (see config.h.in). <blockquote>#ifdef HAVE_CONFIG_H #include <config.h> #endif </blockquote> which contains definitions such as: <blockquote>/* Enable GNU extensions on systems that have them. */ #ifndef _GNU_SOURCE # define _GNU_SOURCE 1 #endif /* Enable threading extensions on Solaris. */ #ifndef _POSIX_PTHREAD_SEMANTICS # define _POSIX_PTHREAD_SEMANTICS 1 #endif /* Enable extensions on HP NonStop. */ #ifndef _TANDEM_SOURCE # define _TANDEM_SOURCE 1 #endif /* Enable general extensions on Solaris. */ #ifndef __EXTENSIONS__ # define __EXTENSIONS__ 1 #endif </blockquote> and so on... When a module is configured (running ./configure), the appropriate values are filled-in based on what the platform is. If hard coded in the C code, all this work has to be done all over again. In addition, this mechanism was put in place because the gcc command might be lengthy on some platforms. Check it out, it should solve the problem. </body> </html>