Another dead configure test - Mailing list pgsql-hackers

From Tom Lane
Subject Another dead configure test
Date
Msg-id 2442359.1660835043@sss.pgh.pa.us
Whole thread Raw
Responses Re: Another dead configure test
List pgsql-hackers
I happened to notice that configure extracts TCL_SHLIB_LD_LIBS
from tclConfig.sh, and puts the value into Makefile.global,
but then we never use it anywhere.  AFAICT the only use went
away in cd75f94da, in 2003.  I propose the attached.

            regards, tom lane

diff --git a/configure b/configure
index b7fd6c5f4e..9e8ac510ac 100755
--- a/configure
+++ b/configure
@@ -634,7 +634,6 @@ DBTOEPUB
 FOP
 XSLTPROC
 XMLLINT
-TCL_SHLIB_LD_LIBS
 TCL_SHARED_BUILD
 TCL_LIB_SPEC
 TCL_LIBS
@@ -18818,7 +18817,7 @@ eval TCL_LIBS=\"$TCL_LIBS\"
 eval TCL_LIB_SPEC=\"$TCL_LIB_SPEC\"
 eval TCL_SHARED_BUILD=\"$TCL_SHARED_BUILD\"

-        if test "$TCL_SHARED_BUILD" != 1; then
+    if test "$TCL_SHARED_BUILD" != 1; then
       as_fn_error $? "cannot build PL/Tcl because Tcl is not a shared library
 Use --without-tcl to disable building PL/Tcl." "$LINENO" 5
     fi
diff --git a/configure.ac b/configure.ac
index e5740f4fb5..67cf317c3b 100644
--- a/configure.ac
+++ b/configure.ac
@@ -2252,7 +2252,6 @@ if test "$with_tcl" = yes; then
     PGAC_PATH_TCLCONFIGSH([$with_tclconfig])
     PGAC_EVAL_TCLCONFIGSH([$TCL_CONFIG_SH],
                           [TCL_INCLUDE_SPEC,TCL_LIBS,TCL_LIB_SPEC,TCL_SHARED_BUILD])
-    AC_SUBST(TCL_SHLIB_LD_LIBS)dnl don't want to double-evaluate that one
     if test "$TCL_SHARED_BUILD" != 1; then
       AC_MSG_ERROR([cannot build PL/Tcl because Tcl is not a shared library
 Use --without-tcl to disable building PL/Tcl.])
diff --git a/src/Makefile.global.in b/src/Makefile.global.in
index 0625b60c43..5664c645f8 100644
--- a/src/Makefile.global.in
+++ b/src/Makefile.global.in
@@ -221,7 +221,6 @@ TCL_LIBS        = @TCL_LIBS@
 TCL_LIB_SPEC        = @TCL_LIB_SPEC@
 TCL_INCLUDE_SPEC    = @TCL_INCLUDE_SPEC@
 TCL_SHARED_BUILD    = @TCL_SHARED_BUILD@
-TCL_SHLIB_LD_LIBS    = @TCL_SHLIB_LD_LIBS@

 PTHREAD_CFLAGS        = @PTHREAD_CFLAGS@
 PTHREAD_LIBS        = @PTHREAD_LIBS@

pgsql-hackers by date:

Previous
From: Andrew Dunstan
Date:
Subject: Re: Add support for DEFAULT specification in COPY FROM
Next
From: Robert Haas
Date:
Subject: Re: [Proposal] Fully WAL logged CREATE DATABASE - No Checkpoints