portability fix in BUILD/* for solaris

BUILD/check-cpu

@@ -3,9 +3,9 @@
 # Check cpu of current machine and find the
 # best compiler optimization flags for gcc
 #
-#
 
-if test -r /proc/cpuinfo ; then
+check_cpu () {
+  if test -r /proc/cpuinfo ; then
     # on Linux (and others?) we can get detailed CPU information out of /proc
     cpuinfo="cat /proc/cpuinfo"
 
@@ -31,7 +31,7 @@ if test -r /proc/cpuinfo ; then
     for flag in `$cpuinfo | grep '^flags' | sed -e 's/^flags.*: //'`; do 
           eval cpu_flag_$flag=yes
     done
-else
+  else
     # Fallback when there is no /proc/cpuinfo
     case "`uname -s`" in
       FreeBSD|OpenBSD)
@@ -47,12 +47,12 @@ else
         model_name=`uname -p`;
         ;;
     esac
-fi
+  fi
 
-# detect CPU shortname as used by gcc options 
-# this list is not complete, feel free to add further entries
-cpu_arg=""
-case "$cpu_family--$model_name" in
+  # detect CPU shortname as used by gcc options 
+  # this list is not complete, feel free to add further entries
+  cpu_arg=""
+  case "$cpu_family--$model_name" in
     # DEC Alpha
     Alpha*EV6*)
       cpu_arg="ev6";
@@ -110,27 +110,27 @@ case "$cpu_family--$model_name" in
     *)
       cpu_arg="";
       ;;
-esac
+  esac
 
 
-if test -z "$cpu_arg"; then
-  echo "BUILD/check-cpu: Oops, could not find out what kind of cpu this machine is using."
+  if test -z "$cpu_arg"; then
+    echo "BUILD/check-cpu: Oops, could not find out what kind of cpu this machine is using." >&2
     check_cpu_cflags=""
     return
-fi
+  fi
 
-# different compiler versions have different option names
-# for CPU specific command line options
-if test -z "$CC" ; then
+  # different compiler versions have different option names
+  # for CPU specific command line options
+  if test -z "$CC" ; then
     cc="gcc";
-else
+  else
     cc=$CC
-fi
+  fi
 
-cc_ver=`$cc --version | sed 1q`
-cc_verno=`echo $cc_ver | sed -e 's/[^0-9. ]//g;	 s/^ *//g; s/ .*//g'`
+  cc_ver=`$cc --version | sed 1q`
+  cc_verno=`echo $cc_ver | sed -e 's/[^0-9. ]//g;	 s/^ *//g; s/ .*//g'`
 
-case "$cc_ver--$cc_verno" in
+  case "$cc_ver--$cc_verno" in
     *GCC*)
         # different gcc backends (and versions) have different CPU flags
         case `gcc -dumpmachine` in
@@ -161,22 +161,23 @@ case "$cc_ver--$cc_verno" in
       check_cpu_cflags=""
       return
       ;;
-esac
+  esac
 
-# now we check whether the compiler really understands the cpu type
-touch __test.c
+  # now we check whether the compiler really understands the cpu type
+  touch __test.c
 
-while [ "$cpu_arg" ] ; do
-  echo -n testing $cpu_arg "... "
+  while [ "$cpu_arg" ] ; do
+    # FIXME: echo -n isn't portable - see contortions autoconf goes through
+    echo -n testing $cpu_arg "... " >&2
           
     # compile check
     check_cpu_cflags=`eval echo $check_cpu_args`
     if $cc -c $check_cpu_cflags __test.c 2>/dev/null; then
-	  echo ok
+            echo ok >&2
         break;
     fi
 
-  echo failed
+    echo failed >&2
     check_cpu_cflags=""
 
     # if compile failed: check whether it supports a predecessor of this CPU
@@ -199,7 +200,9 @@ while [ "$cpu_arg" ] ; do
 
       *)          cpu_arg=""          ;;
     esac
-done
+  done
 
-rm __test.*
+  rm __test.*
+}
 
+check_cpu