configure.ac should remove tclsh8.7
(1) By anticrisis on 2021-01-10 22:08:14 [link] [source]
From the release branch, I suggest removing tclsh8.7 from configure.ac and regenerating configure.
The current version of tclsh8.7 uses a virtual filesystem. This will confuse SQLite's build system, because the build system invokes TCLSH_CMD to determine the install directory for Tcl files. It should not do this, but it does, and this will no longer work with tclsh8.7, because 8.7 will return a virtual path which is meaningless to the unix shell.
--- configure.ac
+++ configure.ac
@@ -118,11 +118,11 @@
#########
# See whether we can run specific tclsh versions known to work well;
# if not, then we fall back to plain tclsh.
# TODO: try other versions before falling back?
#
-AC_CHECK_PROGS(TCLSH_CMD, [tclsh8.7 tclsh8.6 tclsh8.5 tclsh], none)
+AC_CHECK_PROGS(TCLSH_CMD, [tclsh8.6 tclsh8.5 tclsh], none)
if test "$TCLSH_CMD" = "none"; then
# If we can't find a local tclsh, then building the amalgamation will fail.
# We act as though --disable-amalgamation has been used.
echo "Warning: can't find tclsh - defaulting to non-amalgamation build."
USE_AMALGAMATION=0
(2) By Dan Kennedy (dan) on 2021-01-12 14:27:14 in reply to 1 [link] [source]
(3) By anticrisis on 2021-01-15 20:06:01 in reply to 2 [source]
Thank you, that's a better solution and it does the trick. I'm sorry for the delay, I didn't get a notification that you had replied.