On Mon, Feb 24, 2025 at 06:45:21PM +0100, Volker Schlecht wrote:
> To have the (most probably) fixed diff out there ...

This is the diff I tested in a bulk. I don't know enugh about sqlite
to have a real opinion here, but this does look harmless.

Do you know why this isn't enabled by default?

> Index: Makefile
> ===================================================================
> RCS file: /cvs/ports/databases/sqlite3/Makefile,v
> diff -u -p -r1.134 Makefile
> --- Makefile	30 Jan 2025 08:16:03 -0000	1.134
> +++ Makefile	24 Feb 2025 17:43:07 -0000
> @@ -2,10 +2,10 @@ COMMENT=	embedded SQL implementation
>  
>  DISTNAME =	sqlite-autoconf-3480000
>  PKGNAME=	sqlite3-3.48.0
> -REVISION=	0
> +REVISION=	1
>  
>  # XXX needs bumps every time :-
> -SHARED_LIBS +=  sqlite3                   37.30 # 8.6
> +SHARED_LIBS +=  sqlite3                   37.31 # 8.6
>  # sqlite suggests that users might like to assert() that library and header
>  # versions match, so bumps are needed even if function signatures don't change.
>  # ... at the current time the only one noticed is a < check (in subversion)
> @@ -35,7 +35,9 @@ CONFIGURE_ARGS +=	--enable-fts5
>  CFLAGS+=	-DSQLITE_ENABLE_UNLOCK_NOTIFY \
>  		-DSQLITE_ENABLE_FTS3 \
>  		-DSQLITE_ENABLE_DBSTAT_VTAB \
> -		-DSQLITE_ENABLE_COLUMN_METADATA=1
> +		-DSQLITE_ENABLE_COLUMN_METADATA=1 \
> +		-DSQLITE_ENABLE_SESSION \
> +		-DSQLITE_ENABLE_PREUPDATE_HOOK
>  
>  BUILD_DEPENDS =	converters/sqlite2mdoc
>  post-install: