(1) g++ 4.x (or Mac OS X) (2) Solaris g++ or CC

Message-ID: <45083D88.7050207@iki.fi>

Plus a tweak to the name of CC.

p4raw-id: //depot/perl@28842

1 files changed, 20 insertions, 1 deletions

diff --git a/cflags.SH b/cflags.SH
index cc369d9ccd..966901a935 100755
--- a/cflags.SH
+++ b/cflags.SH
@@ -225,7 +225,26 @@ for file do
     esac
 
 case "$cc" in
-*g++*) ccflags="`echo $ccflags|sed 's/-Wdeclaration-after-statement/ /'`" ;;
+*g++*)
+  # We need to remove this also in here (removed early earlier).
+  ccflags="`echo $ccflags|sed 's/-Wdeclaration-after-statement/ /'`"
+  ;;
+esac
+
+case "$cc" in
+*g++*)
+  # Without -Wno-unused-variable g++ 4.x compiles are rather unwatchable
+  # because of all the warnings about Perl___notused, and g++ doesn't
+  # do __attribute__((unused)) (and even if at some stage it will), people
+  # do have older gcc installations.  Yes, we lose some valid warnings.
+  for o in -Wno-unused-variable
+  do
+    case "$warn" in
+    *$o*) ;;
+    *) warn="$warn $o" ;;
+    esac
+  done
+  ;;
 esac
 
 if test -f .patch; then