SQLite Forum

configure.ac should remove tclsh8.7

configure.ac should remove tclsh8.7

(1) By anticrisis on 2021-01-10 22:08:14 [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."

(2) By Dan Kennedy (dan) on 2021-01-12 14:27:14 in reply to 1 [link] [source]

Does it work for you as of this commit?




(3) By anticrisis on 2021-01-15 20:06:01 in reply to 2 [link] [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.