From 1ed0387476138f83ba173d161a54b779353d11aa Mon Sep 17 00:00:00 2001
From: "H. Peter Anvin" <hpa@zytor.com>
Date: Sun, 15 Apr 2007 22:45:25 +0000
Subject: [PATCH] Cleaner way to add gcc options

We can actually test for the options being accepted, rather than try
to test for gcc; this handles differences between gcc versions as well
as compilers with a similar command line set.
---
 aclocal.m4   | 15 +++++++++++++++
 configure.in | 11 +++++------
 2 files changed, 20 insertions(+), 6 deletions(-)
 create mode 100644 aclocal.m4

diff --git a/aclocal.m4 b/aclocal.m4
new file mode 100644
index 00000000..e25d13d3
--- /dev/null
+++ b/aclocal.m4
@@ -0,0 +1,15 @@
+dnl --------------------------------------------------------------------------
+dnl PA_ADD_CFLAGS()
+dnl
+dnl Attempt to add the given option to CFLAGS, if it doesn't break compilation
+dnl --------------------------------------------------------------------------
+AC_DEFUN(PA_ADD_CFLAGS,
+[AC_MSG_CHECKING([if $CC accepts $1])
+ pa_add_cflags__old_cflags="$CFLAGS"
+ CFLAGS="$CFLAGS $1"
+ AC_TRY_LINK([#include <stdio.h>],
+ [printf("Hello, World!\n");],
+ AC_MSG_RESULT([yes]),
+ AC_MSG_RESULT([no])
+ CFLAGS="$pa_add_cflags__old_cflags")])
+
diff --git a/configure.in b/configure.in
index 74e784c5..b9d091af 100644
--- a/configure.in
+++ b/configure.in
@@ -59,12 +59,11 @@ else
 fi
 AC_PROG_INSTALL
 
-if test "$GCC" = "yes"; then
-  GCCFLAGS="-W -Wall -std=c99 -pedantic"
-else
-  GCCFLAGS=
-fi
-AC_SUBST(GCCFLAGS)
+dnl If we have gcc, add appropriate options
+PA_ADD_CFLAGS([-W])
+PA_ADD_CFLAGS([-Wall])
+PA_ADD_CFLAGS([-std=c99])
+PA_ADD_CFLAGS([-pedantic])
 
 dnl Look for "nroff" or "groff"
 AC_CHECK_PROGS(NROFF, nroff, echo)