From: Yves Orton <demerphq@dromedary.booking.com>
Date: Sun, 21 Dec 2008 17:38:46 +0000 (+0100)
Subject: fix broken indentation, and move PERL_PATCHNUM logic to a different place so it only... 
X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=commitdiff_plain;h=a7d5944154465240881e85b933688191ffea3b77;p=p5sagit%2Fp5-mst-13.2.git

fix broken indentation, and move PERL_PATCHNUM logic to a different place so it only affects perl.c
---

diff --git a/Makefile.SH b/Makefile.SH
index 5040e53..960f776 100644
--- a/Makefile.SH
+++ b/Makefile.SH
@@ -540,7 +540,9 @@ sperl.i: perl.c $(h)
 make_patchnum:
 	sh $(shellflags) make_patchnum.sh
 
-perl$(OBJ_EXT): make_patchnum .patchnum
+.patchnum: make_patchnum
+
+perl$(OBJ_EXT): .patchnum
 
 translators:	miniperl$(EXE_EXT) $(CONFIGPM) FORCE
 	@echo " "; echo "	Making x2p stuff"; cd x2p; $(LDLIBPTH) $(MAKE) all
diff --git a/cflags.SH b/cflags.SH
index 2546586..8a4008b 100755
--- a/cflags.SH
+++ b/cflags.SH
@@ -296,7 +296,11 @@ for file do
     op) ;;
     opmini) ;;
     pad) ;;
-    perl) ;;
+    perl) 
+        if [ -f .patchnum -a -n "$(cat .patchnum)" ] ; then
+                ccflags="-DPERL_PATCHNUM=`cat .patchnum` $ccflags"
+        fi
+        ;;
     perlapi) ;;
     perlmain) ;;
     perly) ;;
@@ -321,42 +325,39 @@ for file do
     *) ;;
     esac
 
-case "$cc" in
-*g++*)
-  # Extra paranoia in case people have bad canned ccflags:
-  # bad in the sense that the flags are accepted by g++,
-  # but then whined about.
-  for f in -Wdeclaration-after-statement -std=c89
-  do
-    ccflags=`echo $ccflags|sed 's/$f/ /'`
-  done
-  ;;
-esac
-cppflags=`echo $cppflags|sed 's/-Wdeclaration-after-statement/ /'`
-
-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 may, people do
-  # have older gcc installations), and ((void)x) isn't enough to silence
-  # the noises about XS functions not using their cv parameter, so we need
-  # the -Wno-unused-parameter too.
-  # Yes, we lose some valid warnings, but hopefully other compilers
-  # (like gcc) will still pick up those warnings.
-  for o in -Wno-unused-variable -Wno-unused-parameter
-  do
-    case "$warn" in
-    *$o*) ;;
-    *) warn="$warn $o" ;;
+    case "$cc" in
+    *g++*)
+      # Extra paranoia in case people have bad canned ccflags:
+      # bad in the sense that the flags are accepted by g++,
+      # but then whined about.
+      for f in -Wdeclaration-after-statement -std=c89
+      do
+        ccflags=`echo $ccflags|sed 's/$f/ /'`
+      done
+      ;;
+    esac
+    cppflags=`echo $cppflags|sed 's/-Wdeclaration-after-statement/ /'`
+
+    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 may, people do
+      # have older gcc installations), and ((void)x) isn't enough to silence
+      # the noises about XS functions not using their cv parameter, so we need
+      # the -Wno-unused-parameter too.
+      # Yes, we lose some valid warnings, but hopefully other compilers
+      # (like gcc) will still pick up those warnings.
+      for o in -Wno-unused-variable -Wno-unused-parameter
+      do
+        case "$warn" in
+        *$o*) ;;
+        *) warn="$warn $o" ;;
+        esac
+      done
+      ;;
     esac
-  done
-  ;;
-esac
 
-if [ -f .patchnum -a -n "$(cat .patchnum)" ] ; then
-    ccflags="-DPERL_PATCHNUM=`cat .patchnum`"
-fi
 
     : Can we perhaps use $ansi2knr here
     echo "$cc -c -DPERL_CORE $ccflags $stdflags $optimize $warn $extra"
@@ -364,6 +365,7 @@ fi
 
     . $TOP/config.sh
 
+    # end per file behaviour
 done
 !NO!SUBS!
 chmod 755 cflags