X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=cflags.SH;h=1bd5c947f4e265e7fbe0f9e73012491d10645e0a;hb=d308175ab83748573482cc00d7ded514f6a34f91;hp=d2564afe96ec3bfc632e3e583308b15eec8dd2b1;hpb=a6ec74c1448e028e8d796742c81e78fb067bf603;p=p5sagit%2Fp5-mst-13.2.git

diff --git a/cflags.SH b/cflags.SH
index d2564af..1bd5c94 100755
--- a/cflags.SH
+++ b/cflags.SH
@@ -42,6 +42,13 @@ case $PERL_CONFIG_SH in
 	;;
 esac
 
+case "X$1" in
+Xoptimize=*|X"optimize=*")
+	eval "$1"
+	shift
+	;;
+esac
+
 perltype=''
 optdebug=''	# ensure -g used if building a -DDEBUGGING libperl
 case $# in
@@ -131,9 +138,25 @@ for file do
 		optimize="$optdebug"
 	fi
 
+# Add -Wall for the core and core modules iff gcc and not already -Wall
+warn=''
+case "$gccversion" in
+'') ;;
+*)  case "$ccflags" in
+    *-Wall*) ;;
+# Can't add -ansi here because it will fail e.g. in Solaris.
+# Off_t/off_t is a struct in Solaris with largefiles, and with -ansi
+# that struct cannot be compared with a flat integer, such as a STRLEN.
+# The -ansi will also cause a lot of noise in Solaris because of:
+# /usr/include/sys/resource.h:148: warning: `struct rlimit64' declared inside parameter list
+    *) warn='-Wall' ;;
+    esac
+    ;;
+esac
+
     : Can we perhaps use $ansi2knr here
-    echo "$cc -c -DPERL_CORE $ccflags $optimize $perltype"
-    eval "$also "'"$cc -DPERL_CORE -c $ccflags $optimize $perltype"'
+    echo "$cc -c -DPERL_CORE $ccflags $optimize $warn $perltype"
+    eval "$also "'"$cc -DPERL_CORE -c $ccflags $optimize $warn $perltype"'
 
     . $TOP/config.sh