- (tim) [configure.ac ] Not all gcc's support -Wsign-compare

diff --git a/ChangeLog b/ChangeLog
index dd8a04e11e0ffb8abb7664094a3160145d0e91dd..f52e7f16c3ab7c9cf3d0a4f9f6a057486c74cee0 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -2,6 +2,7 @@
  - (dtucker) [regress/test-exec.sh] Do not prepend an extra "/" to a fully-
    qualified sshd pathname since some systems (eg Cygwin) may consider "/foo"
    and "//foo" to be different.  Spotted by vinschen at redhat.com.
+ - (tim) [configure.ac ] Not all gcc's support -Wsign-compare
 
 20050821
  - (dtucker) [configure.ac defines.h includes.h sftp.c] Add support for

diff --git a/configure.ac b/configure.ac
index 5b1ad46774892290ecd2bf778c48d46518138a18..eea151fe8bdc4f7762b934104b300a0f74a82aa5 100644 (file)
--- a/configure.ac
+++ b/configure.ac
@@ -82,7 +82,13 @@ AC_CHECK_DECL(LLONG_MAX, have_llong_max=1, , [#include <limits.h>])
 
 if test "$GCC" = "yes" || test "$GCC" = "egcs"; then
        CFLAGS="$CFLAGS -Wall -Wpointer-arith -Wuninitialized"
-       CFLAGS="$CFLAGS -Wsign-compare"
+       GCC_VER=`$CC --version`
+       case $GCC_VER in
+               1.*) ;;
+               2.8* | 2.9*) CFLAGS="$CFLAGS -Wsign-compare" ;;
+               2.*) ;;
+               *) CFLAGS="$CFLAGS -Wsign-compare" ;;
+       esac
 
        if test -z "$have_llong_max"; then
                # retry LLONG_MAX with -std=gnu99, needed on some Linuxes