Stuart Henderson wrote:
On 2026/06/11 16:55, Stuart Cassoff wrote:
On Thu, 11 Jun 2026, Chris Billington wrote:

The present databases/sqlite3-tcl port is built with MODTCL_VERSION = 8.6,
producing /usr/local/lib/tcl/sqlite3/libsqlite3513.so


With recent Tcl build systems and Tcl 9 we can build Tcl 8 extensions.
This gives everything in one package, with the downside of two Tcl deps.


Stu



diff -Nurp /usr/ports/databases/sqlite3-tcl/Makefile ./Makefile
--- /usr/ports/databases/sqlite3-tcl/Makefile   Thu May 14 22:51:47 2026
+++ ./Makefile  Thu Jun 11 09:30:11 2026
@@ -1,5 +1,7 @@
  COMMENT =             Tcl bindings for SQLite3
+REVISION = 0
+
  V =                   3.53.0
DISTNAME = sqlite${V}
@@ -19,11 +21,13 @@ SITES =                     ${SITE_SOURCEFORGE:=tcl/}
  DISTFILES =           tclsqlite{sqlite}${V}${EXTRACT_SUFX}
MODULES = lang/tcl
-MODTCL_VERSION =       8.6
+MODTCL_VERSION =       9
BUILD_DEPENDS = ${MODTCL_BUILD_DEPENDS}
-RUN_DEPENDS =          ${MODTCL_RUN_DEPENDS}
+RUN_DEPENDS = ${MODTCL_RUN_DEPENDS} \
+                       lang/tcl/8.6
+
  FAKE_FLAGS =          PKG_DIR=sqlite3
  SEPARATE_BUILD =      Yes
  CONFIGURE_STYLE =     gnu
@@ -45,5 +49,13 @@ VER =                        ${V:S/.//g}
pre-configure:
        @${MODTCL_TCLSH_ADJ} ${WRKSRC}/compat/sqlite3/spaceanal.tcl
+
+post-install:
+       cd ${WRKBUILD} && umask 022 && exec \
+       env -i ${MAKE_ENV} ${MAKE_PROGRAM} ${MAKE_FLAGS} \
+       -f ${MAKE_FILE} clean ${ALL_TARGET} \
+       CPPFLAGS='-DTCL_MAJOR_VERSION=8' \
+       PKG_LIB_FILE=libsqlite${VER}.so
+       ${INSTALL_DATA} ${WRKBUILD}/libsqlite${VER}.so 
${DESTDIR}${MODTCL_TCLDIR}/sqlite3/

I'm not the biggest fan of a build stage in post-install, but it's
probably better than other options here.

nitpicking, but could you indent the lines after \ please?

post-install:
        cd ${WRKBUILD} && umask 022 && exec \
                env -i ${MAKE_ENV} ${MAKE_PROGRAM} ${MAKE_FLAGS} \
                -f ${MAKE_FILE} clean ${ALL_TARGET} \
                CPPFLAGS='-DTCL_MAJOR_VERSION=8' \
                PKG_LIB_FILE=libsqlite${VER}.so
        ${INSTALL_DATA} ${WRKBUILD}/libsqlite${VER}.so 
${DESTDIR}${MODTCL_TCLDIR}/sqlite3/

ok

If the formatting could be corrected, would anyone like to give a second OK?

Chris

Reply via email to