Re: python support in CVS.

[Carey Evans <c.evans@xxxxxxxxxxxx>]

I've tried out the Python support, and I've found some problems with
the configure.in code.  Some of this might be because my /bin/sh is
ash, not bash.

When running configure, I get the following output:

checking for python... /usr/bin/python
checking for python prefix... Traceback (innermost last):
  File "<stdin>", line 3, in ?
NameError: EOF
/usr
checking for python version... Traceback (innermost last):
  File "<stdin>", line 3, in ?
NameError: EOF
1.5

Part of the problem is that the <<EOF isn't being recognised by the
shell.  There's also that standard error should be getting redirected
to config.log.

Instead of the quite complicated code to work out the Python version,
I'd suggest:

  PYTHON_VERSION="`$PYTHON -c 'import sys; print sys.version[[:3]]' 2>&5`"

with similar code for PYTHON_PREFIX.

Looking at the configure.in, I also wonder why the glib.m4 code that
comes with glib isn't being used to set CFLAGS and LIBS, making it a
lot simpler.  This patch makes both these changes.

--- configure.in.dist   Sun Nov 26 21:41:45 2000
+++ configure.in        Sun Nov 26 21:42:23 2000
@@ -18,18 +18,9 @@
 dnl *
 dnl * Not using gnome, so look for glib and GTK+ instead.
 dnl *
-AC_PATH_PROG(GLIB_CONFIG,glib-config,no)
-if test x$GLIB_CONFIG = xno ; then
-       AC_MSG_ERROR(** Can't find glib-config.  Is glib installed? **)
-fi
-AC_MSG_CHECKING(for glib C flags)
-GLIB_CFLAGS="`$GLIB_CONFIG --cflags glib`"
+AM_PATH_GLIB
 CFLAGS="$CFLAGS $GLIB_CFLAGS"
-AC_MSG_RESULT($GLIB_CFLAGS)
-AC_MSG_CHECKING(for glib libs)
-GLIB_LIBS="`$GLIB_CONFIG --libs glib`"
 LIBS="$LIBS $GLIB_LIBS"
-AC_MSG_RESULT($GLIB_LIBS)
 
 dnl *
 dnl * Check for SWIG
@@ -44,22 +35,12 @@
 AC_PATH_PROG(PYTHON,python,no)
 if test x$PYTHON != xno ; then
   AC_MSG_CHECKING(for python prefix)
-changequote({,})dnl
-  PYTHON_PREFIX="`$PYTHON <<EOF
-import sys
-print sys.exec_prefix
-EOF`"
-changequote([,])dnl
+  PYTHON_PREFIX="`$PYTHON -c 'import sys; print sys.exec_prefix' 2>&5`"
   AC_SUBST(PYTHON_PREFIX)
   AC_MSG_RESULT($PYTHON_PREFIX)
 
   AC_MSG_CHECKING(for python version)
-changequote({,})dnl
-  PYTHON_VERSION="`$PYTHON <<EOF
-import sys
-print sys.version[:3]
-EOF`"
-changequote([,])dnl
+  PYTHON_VERSION="`$PYTHON -c 'import sys; print sys.version[[:3]]' 2>&5`"
   AC_SUBST(PYTHON_VERSION)
   AC_MSG_RESULT($PYTHON_VERSION)
 fi


-- 
         Carey Evans  http://home.clear.net.nz/pages/c.evans/
+---
| This is the LINUX5250 Mailing List!
| To submit a new message, send your mail to LINUX5250@midrange.com.
| To subscribe to this list send email to LINUX5250-SUB@midrange.com.
| To unsubscribe from this list send email to LINUX5250-UNSUB@midrange.com.
| Questions should be directed to the list owner/operator: david@midrange.com
+---