]> git.ipfire.org Git - thirdparty/sarg.git/commitdiff
Remove the obsolete cc option
authorFrédéric Marchal <fmarchal@users.sourceforge.net>
Wed, 1 Feb 2012 10:34:45 +0000 (11:34 +0100)
committerFrédéric Marchal <fmarchal@users.sourceforge.net>
Wed, 1 Feb 2012 10:34:45 +0000 (11:34 +0100)
If sarg is compiled with cc, the old configure script would add the -Aa
option to the CFLAGS but that option produces an error on modern cc. The
error is about a missing argument. I could not find the new syntax nor the
purpose of that option but it was reported that removing the whole
condition was solving the problem.

Bug report here:

http://sourceforge.net/tracker/?func=detail&aid=3467463&group_id=68910&atid=522791

configure.in

index 33aa9d2466354c057d31d01150eb1967776b24e1..a5732e0048c70332c04485b5d8753aa020460a81 100644 (file)
@@ -17,9 +17,13 @@ dnl yes ) CC=g++ ;;
 dnl  *   ) AC_PROG_CXX ;;
 esac
 
-if test "${CC}" = "cc" ; then
-   CFLAGS="${CFLAGS} -g -O2 -Aa"
-fi
+dnl The purpose of the following condition is unknown but it is obsolete as it is.
+dnl The -Aa option produces an error claiming that some argument are missing.
+dnl if test "${CC}" = "cc" ; then
+dnl    CFLAGS="${CFLAGS} -g -O2 -Aa"
+dnl fi
+
+# Report more warnings to improve code quality.
 CFLAGS="${CFLAGS} -Wall -Wno-sign-compare"
 
 dnl Check for supported compiler options