Re: Removing built-in support for sqlite3

[Greg Troxel <gdt%lexort.com@localhost>]

David Holland <dholland-pkgtech%netbsd.org@localhost> writes:

> On Sat, Jul 20, 2024 at 08:27:26AM +0200, Thomas Klausner wrote:
>  > NetBSD comes with sqlite3 in the base system. It mostly imported for
>  > supporting a heimdal feature (since disabled) and for apropos(1).
>  > 
>  > The version is not kept up-to-date (NetBSD 10 comes with 3.26, pkgsrc
>  > has 3.45). Some software in pkgsrc requires features that are not
>  > enabled in the base system version.
>  > 
>  > I'd like to remove builtin support for sqlite3 from pkgsrc (basically,
>  > cvs rm databases/sqlite3/builtin.mk).
>
> Is there any reason to do that vs. just setting BUILDLINK_API_DEPENDS
> so the version in question isn't used?

The problem is that many packages depend on sqlite3, and having two
versions in the mix means you have a risk of linking both the base
version and the pkgsrc version.  We have multiple packages that are not
ok with the base version (matrix-synapse, because of missing features,
and spatialite, not sure why but doesn't seem wrong).

The only way out that I can see, is to have only one version, and have
it be up to date.   Thus, on any given system, either:
  - banning base sqlite
  - deciding that base is ok and never using pkgsrc, causing any package
    that wants better than base to fail

and all in all, the first option seems like a better outcome, easier,
and more resistant to future breakage.