Re: [HACKERS] mega-patch--tcl/tk configuration

Marc wrote:

| Having it search several directories in order of preference, IMHO, is
| proper behaviour for configure...it should remain as version search
| instead of "just tcl.h"...

Bruce answered:

| OK, go ahead.  I did fix several problem with the tcl/tk stuff also.
| For example, if it found /usr/include/tk.h, the flag would be -I with no
| directory, causing any compile to fail.

While you guys are thinking about this, let me remind you of the
following patch for these problems that I sent you on March 2nd.  It
fixes the problem Bruce mentions, and extends the version checking to
include the last versions of TCL and TK prior to 8.0 -- since I can
observe here that they do work.

Marc's argument that version numbers have to stay where TCL/TK is
involved is very important: they are nearly always incompatible with
themselves in preceding and following versions, making it necessary
for a typical site to have one complete TCL/TK installation for each
and every application that uses the damn things.  (You may recall that
I expressed amazement at my discovery that I could use 7.6/4.2 with
both Scotty and PostgreSQL.)

Anyway, my take on the --with-tcl (and --with-perl) things, relative
to PostgreSQL 6.3:

*** configure.in.orig    Mon Mar  2 06:33:14 1998
--- configure.in    Mon Mar  2 21:42:59 1998
***************
*** 239,246 ****
  AC_ARG_WITH(
     tcl,
     [   --with-tcl              use tcl ],
!    USE_TCL=true AC_MSG_RESULT(enabled),
!    USE_TCL=false AC_MSG_RESULT(disabled)
  )
  export USE_TCL
  USE_X=$USE_TCL
--- 239,246 ----
  AC_ARG_WITH(
     tcl,
     [   --with-tcl              use tcl ],
!    USE_TCL=true; AC_MSG_RESULT(enabled),
!    USE_TCL=false; AC_MSG_RESULT(disabled)
  )
  export USE_TCL
  USE_X=$USE_TCL
***************
*** 250,257 ****
  AC_ARG_WITH(
     perl,
     [   --with-perl             use perl ],
!    USE_PERL=true AC_MSG_RESULT(enabled),
!    USE_PERL=false AC_MSG_RESULT(disabled)
  )
  export USE_PERL

--- 250,257 ----
  AC_ARG_WITH(
     perl,
     [   --with-perl             use perl ],
!    USE_PERL=true; AC_MSG_RESULT(enabled),
!    USE_PERL=false; AC_MSG_RESULT(disabled)
  )
  export USE_PERL

***************
*** 563,570 ****
  if test "$USE_TCL" = "true"
  then
  TCL_INCDIR=no
! AC_CHECK_HEADER(tcl.h, TCL_INCDIR=)
! for f in /usr/include /usr/include/tcl8.0 /usr/local/include /usr/local/include/tcl8.0; do
  if test "$TCL_INCDIR" = "no"; then
  AC_CHECK_HEADER($f/tcl.h, TCL_INCDIR=$f)
  fi
--- 563,569 ----
  if test "$USE_TCL" = "true"
  then
  TCL_INCDIR=no
! for f in /usr/include /usr/include/tcl8.0 /usr/include/tcl7.6 /usr/local/include /usr/local/include/tcl8.0
/usr/local/include/tcl7.6;do 
  if test "$TCL_INCDIR" = "no"; then
  AC_CHECK_HEADER($f/tcl.h, TCL_INCDIR=$f)
  fi
***************
*** 580,586 ****
  if test "$USE_TCL" = "true"
  then
  TCL_LIB=
! for f in tcl8.0 tcl80; do
  if test -z "$TCL_LIB"; then
  AC_CHECK_LIB($f, main, TCL_LIB=$f)
  fi
--- 579,585 ----
  if test "$USE_TCL" = "true"
  then
  TCL_LIB=
! for f in tcl8.0 tcl80 tcl7.6 tcl76; do
  if test -z "$TCL_LIB"; then
  AC_CHECK_LIB($f, main, TCL_LIB=$f)
  fi
***************
*** 606,616 ****
  ice_save_CPPFLAGS="$CPPFLAGS"
  ice_save_LDFLAGS="$LDFLAGS"

  CPPFLAGS="$CPPFLAGS $X_CFLAGS -I$TCL_INCDIR"

  TK_INCDIR=no
! AC_CHECK_HEADER(tk.h, TK_INCDIR=)
! for f in /usr/include /usr/include/tk8.0 /usr/local/include /usr/local/include/tk8.0; do
  if test "$TK_INCDIR" = "no"; then
  AC_CHECK_HEADER($f/tk.h, TK_INCDIR=$f)
  fi
--- 605,619 ----
  ice_save_CPPFLAGS="$CPPFLAGS"
  ice_save_LDFLAGS="$LDFLAGS"

+ if test "$TCL_INCDIR" = ""
+ then
+ CPPFLAGS="$CPPFLAGS $X_CFLAGS"
+ else
  CPPFLAGS="$CPPFLAGS $X_CFLAGS -I$TCL_INCDIR"
+ fi

  TK_INCDIR=no
! for f in /usr/include /usr/include/tk8.0 /usr/include/tk4.2 /usr/local/include /usr/local/include/tk8.0
/usr/local/include/tk4.2;do 
  if test "$TK_INCDIR" = "no"; then
  AC_CHECK_HEADER($f/tk.h, TK_INCDIR=$f)
  fi
***************
*** 631,637 ****
  if test "$USE_TCL" = "true"
  then
  TK_LIB=
! for f in tk8.0 tk80; do
  if test -z "$TK_LIB"; then
  AC_CHECK_LIB($f, main, TK_LIB=$f)
  fi
--- 634,640 ----
  if test "$USE_TCL" = "true"
  then
  TK_LIB=
! for f in tk8.0 tk80 tk4.2 tk42; do
  if test -z "$TK_LIB"; then
  AC_CHECK_LIB($f, main, TK_LIB=$f)
  fi
*** configure.orig    Mon Mar  2 15:55:42 1998
--- configure    Mon Mar  2 21:43:39 1998
***************
*** 825,833 ****
  # Check whether --with-tcl or --without-tcl was given.
  if test "${with_tcl+set}" = set; then
    withval="$with_tcl"
!   USE_TCL=true echo "$ac_t""enabled" 1>&6
  else
!   USE_TCL=false echo "$ac_t""disabled" 1>&6

  fi

--- 825,833 ----
  # Check whether --with-tcl or --without-tcl was given.
  if test "${with_tcl+set}" = set; then
    withval="$with_tcl"
!   USE_TCL=true; echo "$ac_t""enabled" 1>&6
  else
!   USE_TCL=false; echo "$ac_t""disabled" 1>&6

  fi

***************
*** 839,847 ****
  # Check whether --with-perl or --without-perl was given.
  if test "${with_perl+set}" = set; then
    withval="$with_perl"
!   USE_PERL=true echo "$ac_t""enabled" 1>&6
  else
!   USE_PERL=false echo "$ac_t""disabled" 1>&6

  fi

--- 839,847 ----
  # Check whether --with-perl or --without-perl was given.
  if test "${with_perl+set}" = set; then
    withval="$with_perl"
!   USE_PERL=true; echo "$ac_t""enabled" 1>&6
  else
!   USE_PERL=false; echo "$ac_t""disabled" 1>&6

  fi

***************
*** 5393,5432 ****
  if test "$USE_TCL" = "true"
  then
  TCL_INCDIR=no
! ac_safe=`echo "tcl.h" | sed 'y%./+-%__p_%'`
! echo $ac_n "checking for tcl.h""... $ac_c" 1>&6
! echo "configure:5399: checking for tcl.h" >&5
! if eval "test \"`echo '$''{'ac_cv_header_$ac_safe'+set}'`\" = set"; then
!   echo $ac_n "(cached) $ac_c" 1>&6
! else
!   cat > conftest.$ac_ext <<EOF
! #line 5404 "configure"
! #include "confdefs.h"
! #include <tcl.h>
! EOF
! ac_try="$ac_cpp conftest.$ac_ext >/dev/null 2>conftest.out"
! { (eval echo configure:5409: \"$ac_try\") 1>&5; (eval $ac_try) 2>&5; }
! ac_err=`grep -v '^ *+' conftest.out`
! if test -z "$ac_err"; then
!   rm -rf conftest*
!   eval "ac_cv_header_$ac_safe=yes"
! else
!   echo "$ac_err" >&5
!   echo "configure: failed program was:" >&5
!   cat conftest.$ac_ext >&5
!   rm -rf conftest*
!   eval "ac_cv_header_$ac_safe=no"
! fi
! rm -f conftest*
! fi
! if eval "test \"`echo '$ac_cv_header_'$ac_safe`\" = yes"; then
!   echo "$ac_t""yes" 1>&6
!   TCL_INCDIR=
! else
!   echo "$ac_t""no" 1>&6
! fi
!
! for f in /usr/include /usr/include/tcl8.0 /usr/local/include /usr/local/include/tcl8.0; do
  if test "$TCL_INCDIR" = "no"; then
  ac_safe=`echo "$f/tcl.h" | sed 'y%./+-%__p_%'`
  echo $ac_n "checking for $f/tcl.h""... $ac_c" 1>&6
--- 5393,5399 ----
  if test "$USE_TCL" = "true"
  then
  TCL_INCDIR=no
! for f in /usr/include /usr/include/tcl8.0 /usr/include/tcl7.6 /usr/local/include /usr/local/include/tcl8.0
/usr/local/include/tcl7.6;do 
  if test "$TCL_INCDIR" = "no"; then
  ac_safe=`echo "$f/tcl.h" | sed 'y%./+-%__p_%'`
  echo $ac_n "checking for $f/tcl.h""... $ac_c" 1>&6
***************
*** 5473,5479 ****
  if test "$USE_TCL" = "true"
  then
  TCL_LIB=
! for f in tcl8.0 tcl80; do
  if test -z "$TCL_LIB"; then
  echo $ac_n "checking for main in -l$f""... $ac_c" 1>&6
  echo "configure:5480: checking for main in -l$f" >&5
--- 5440,5446 ----
  if test "$USE_TCL" = "true"
  then
  TCL_LIB=
! for f in tcl8.0 tcl80 tcl7.6 tcl76; do
  if test -z "$TCL_LIB"; then
  echo $ac_n "checking for main in -l$f""... $ac_c" 1>&6
  echo "configure:5480: checking for main in -l$f" >&5
***************
*** 5531,5573 ****
  ice_save_CPPFLAGS="$CPPFLAGS"
  ice_save_LDFLAGS="$LDFLAGS"

! CPPFLAGS="$CPPFLAGS $X_CFLAGS -I$TCL_INCDIR"
!
! TK_INCDIR=no
! ac_safe=`echo "tk.h" | sed 'y%./+-%__p_%'`
! echo $ac_n "checking for tk.h""... $ac_c" 1>&6
! echo "configure:5540: checking for tk.h" >&5
! if eval "test \"`echo '$''{'ac_cv_header_$ac_safe'+set}'`\" = set"; then
!   echo $ac_n "(cached) $ac_c" 1>&6
! else
!   cat > conftest.$ac_ext <<EOF
! #line 5545 "configure"
! #include "confdefs.h"
! #include <tk.h>
! EOF
! ac_try="$ac_cpp conftest.$ac_ext >/dev/null 2>conftest.out"
! { (eval echo configure:5550: \"$ac_try\") 1>&5; (eval $ac_try) 2>&5; }
! ac_err=`grep -v '^ *+' conftest.out`
! if test -z "$ac_err"; then
!   rm -rf conftest*
!   eval "ac_cv_header_$ac_safe=yes"
! else
!   echo "$ac_err" >&5
!   echo "configure: failed program was:" >&5
!   cat conftest.$ac_ext >&5
!   rm -rf conftest*
!   eval "ac_cv_header_$ac_safe=no"
! fi
! rm -f conftest*
! fi
! if eval "test \"`echo '$ac_cv_header_'$ac_safe`\" = yes"; then
!   echo "$ac_t""yes" 1>&6
!   TK_INCDIR=
  else
!   echo "$ac_t""no" 1>&6
  fi

! for f in /usr/include /usr/include/tk8.0 /usr/local/include /usr/local/include/tk8.0; do
  if test "$TK_INCDIR" = "no"; then
  ac_safe=`echo "$f/tk.h" | sed 'y%./+-%__p_%'`
  echo $ac_n "checking for $f/tk.h""... $ac_c" 1>&6
--- 5498,5512 ----
  ice_save_CPPFLAGS="$CPPFLAGS"
  ice_save_LDFLAGS="$LDFLAGS"

! if test "$TCL_INCDIR" = ""
! then
! CPPFLAGS="$CPPFLAGS $X_CFLAGS"
  else
! CPPFLAGS="$CPPFLAGS $X_CFLAGS -I$TCL_INCDIR"
  fi

! TK_INCDIR=no
! for f in /usr/include /usr/include/tk8.0 /usr/include/tk4.2 /usr/local/include /usr/local/include/tk8.0
/usr/local/include/tk4.2;do 
  if test "$TK_INCDIR" = "no"; then
  ac_safe=`echo "$f/tk.h" | sed 'y%./+-%__p_%'`
  echo $ac_n "checking for $f/tk.h""... $ac_c" 1>&6
***************
*** 5619,5625 ****
  if test "$USE_TCL" = "true"
  then
  TK_LIB=
! for f in tk8.0 tk80; do
  if test -z "$TK_LIB"; then
  echo $ac_n "checking for main in -l$f""... $ac_c" 1>&6
  echo "configure:5626: checking for main in -l$f" >&5
--- 5558,5564 ----
  if test "$USE_TCL" = "true"
  then
  TK_LIB=
! for f in tk8.0 tk80 tk4.2 tk42; do
  if test -z "$TK_LIB"; then
  echo $ac_n "checking for main in -l$f""... $ac_c" 1>&6
  echo "configure:5626: checking for main in -l$f" >&5

-tih
--
Popularity is the hallmark of mediocrity.  --Niles Crane, "Frasier"