pgsql-server/ /configure /configure.in rc/Make ...

Started by Tom Laneover 23 years ago3 messagescomitters
Jump to latest
#1Tom Lane
tgl@sss.pgh.pa.us

CVSROOT: /cvsroot
Module name: pgsql-server
Changes by: tgl@postgresql.org 02/12/30 12:19:56

Modified files:
. : configure configure.in
src : Makefile.global.in
src/bin/pgtclsh: Makefile
src/interfaces/libpgtcl: Makefile
src/pl/tcl : Makefile

Log message:
Make use of TCL_INCLUDE_SPEC if available (it's new in Tcl 8.4, too bad
it took 'em this long to realize it's needed...)

#2Peter Eisentraut
peter_e@gmx.net
In reply to: Tom Lane (#1)
Re: pgsql-server/ /configure /configure.in rc/Make ...

Tom Lane writes:

Log message:
Make use of TCL_INCLUDE_SPEC if available (it's new in Tcl 8.4, too bad
it took 'em this long to realize it's needed...)

Is there a reason why you didn't add TCL_INCLUDE_SPEC to the list in
PGAC_EVAL_TCLCONFIGSH? It seems that that should handle the missing case
as well, since

eval TCL_INCLUDE_SPEC=\"$TCL_INCLUDE_SPEC\"

should be equivalent to

TCL_INCLUDE_SPEC=''

in that case.

--
Peter Eisentraut peter_e@gmx.net

#3Tom Lane
tgl@sss.pgh.pa.us
In reply to: Peter Eisentraut (#2)
Re: pgsql-server/ /configure /configure.in rc/Make ...

Peter Eisentraut <peter_e@gmx.net> writes:

Is there a reason why you didn't add TCL_INCLUDE_SPEC to the list in
PGAC_EVAL_TCLCONFIGSH? It seems that that should handle the missing case
as well, since
eval TCL_INCLUDE_SPEC=\"$TCL_INCLUDE_SPEC\"
should be equivalent to
TCL_INCLUDE_SPEC=''
in that case.

I wasn't sure if that would work portably. If you think it will, feel
free to simplify the code.

regards, tom lane