[PATCH] aclocal.m4: fix WINE_TRY_CFLAGS macro (LLVM/Clang)

Yann Droneaud yann at droneaud.fr
Thu Mar 4 04:43:17 CST 2010


The macro WINE_TRY_CFLAGS try to compile
int main(int argc, char *argv) instead of
int main(int argc, char *argv[])
due to M4 preprocessing and misquoting.

Sadly such prototype for main() is an error
for some compiler such as LLVM/Clang.

Problem fixed by using proper quoting (see
AC_LANG_SOURCE documentation) and using safer
char **argv instead of char *argv[].
---
 aclocal.m4 |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/aclocal.m4 b/aclocal.m4
index 7f5f040..1aba083 100644
--- a/aclocal.m4
+++ b/aclocal.m4
@@ -75,7 +75,7 @@ AC_DEFUN([WINE_TRY_CFLAGS],
 AC_CACHE_CHECK([whether the compiler supports $1], ac_var,
 [ac_wine_try_cflags_saved=$CFLAGS
 CFLAGS="$CFLAGS $1"
-AC_LINK_IFELSE(AC_LANG_SOURCE([int main(int argc, char *argv[]) { return 0; }]),
+AC_LINK_IFELSE(AC_LANG_SOURCE([[int main(int argc, char **argv) { return 0; }]]),
                [AS_VAR_SET(ac_var,yes)], [AS_VAR_SET(ac_var,no)])
 CFLAGS=$ac_wine_try_cflags_saved])
 AS_IF([test AS_VAR_GET(ac_var) = yes],
-- 
1.6.2.5




More information about the wine-patches mailing list