[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[freewnn:00758] Re: recent patches



小野寛生です

aono = aono@cc.osaka-kyoiku.ac.jp (Tomoki AONO) wrote:

aono> `basename $CC`で判定するよりは、AC_PROG_CCで定義される変数
aono> GCC を利用した方がよいと思います。(autoconfのinfo参照) あ

patch つくってみました。どうでしょう。

  if "X$GCC" = Xyes
ではなく、
  case $GCC in
    [Yy][Ee][Ss])
    *)
とかの方がよいのかな。

aono> とはdebugの分岐部分をもう少しスマートにまとめられないか考
aono> え中です。

GCC だった場合の処理を 各 OS 毎の処理とは別にした方がよいのでしょうかね。
Index: configure.in
===================================================================
RCS file: /cvs/FreeWnn/configure.in,v
retrieving revision 1.18
diff -u -r1.18 configure.in
--- configure.in	2001/09/24 08:36:12	1.18
+++ configure.in	2001/09/30 16:17:27
@@ -294,7 +294,7 @@
     fi
     ;;
 *-*-sunos*)
-      if test `basename $CC` = gcc; then
+      if test "X$GCC" = "Xyes" ; then
 	CCOPTIONS="-Dsun -Dsparc"
         if test $debug = true; then
           CDEBUGFLAGS="-g2 -O2"
@@ -308,7 +308,7 @@
       ;;
 *-*-solaris2.*)
     # Will 'solaris*' be OK?
-      if test `basename $CC` = gcc; then
+      if test "X$GCC" = "Xyes" ; then
         CCOPTIONS="-Dsun -D`uname -p` -DSVR4 -DSYSV"
         if test $debug = true; then
           CDEBUGFLAGS="-g2 -O2"
@@ -321,7 +321,7 @@
       fi
     ;;
 *-*-hpux*)
-    if test `basename $CC` = gcc; then
+    if test "X$GCC" = "Xyes" ; then
         CCOPTIONS="-Dhpux -DSYSV"
         if test $debug = true; then
           CDEBUGFLAGS="-g2 -O2"
@@ -349,7 +349,7 @@
         fi
 	;;
 *-*-aix*)
-    if test `basename $CC` = gcc; then
+    if test "X$GCC" = "Xyes" ; then
       CDEBUGFLAGS=-O2
         if test $debug = true; then
 	  CDEBUGFLAGS="-g2"
@@ -365,6 +365,14 @@
     AC_MSG_WARN("It is not reported if FreeWnn works on this system.  But continuing...")
     AC_MSG_WARN([(It may or may not work if you set CCOPTIONS properly.)])
     ;;
+    if test "X$GCC" = "Xyes" ; then
+      CDEBUGFLAGS=-O2
+        if test $debug = true; then
+	  CDEBUGFLAGS="-g2"
+        else
+	  CDEBUGFLAGS=""
+        fi
+    fi
 esac
 
 if test ${debug} = true; then


http://www.freewnn.org/ FreeWnn Project