From 3db1e3fc685cb0c570b4e17a60e0e041fa48b32f Mon Sep 17 00:00:00 2001
From: Tim Rice <tim@multitalents.net>
Date: Tue, 23 Aug 2005 17:11:26 -0700
Subject: [PATCH]  - (tim) [configure.ac ] Not all gcc's support -Wsign-compare

---
 ChangeLog    |  3 ++-
 configure.ac | 10 ++++++++--
 2 files changed, 10 insertions(+), 3 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 5d6982ff4..dd64f1374 100644
--- 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
@@ -2950,4 +2951,4 @@
    - (djm) Trim deprecated options from INSTALL. Mention UsePAM
    - (djm) Fix quote handling in sftp; Patch from admorten AT umich.edu
 
-$Id: ChangeLog,v 1.3876 2005/08/23 13:32:05 dtucker Exp $
+$Id: ChangeLog,v 1.3877 2005/08/24 00:11:26 tim Exp $
diff --git a/configure.ac b/configure.ac
index 619a4e76a..f1588c693 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1,4 +1,4 @@
-# $Id: configure.ac,v 1.287 2005/08/22 22:06:56 dtucker Exp $
+# $Id: configure.ac,v 1.288 2005/08/24 00:11:26 tim Exp $
 #
 # Copyright (c) 1999-2004 Damien Miller
 #
@@ -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