That --with-tcl thing...

Congratulations on a job well done, guys!

Here's a little thing for the first round of patches, though: I have
figured out why --with-tcl didn't work, even after adding the missing
semicolon that was posted about. Here's the patch to configure.in in
version 6.3 that will actually fix it all right (including letting you
use the previous versions of tcl and tk, since they do work). Note
the test for TCL_INCDIR being the empty string, to avoid adding an
empty "-I" to the flags used when checking for tk.h. Note also that
I've removed the simple tests for tcl.h and tk.h without specifying
include directories for them: the actual include directories need to
end up in TCL_INCDIR and TK_INCDIR in Makefile.global, because of the
way they are used in other Makefiles.

*** 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

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