Re: Removing built-in support for sqlite3

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

Taylor R Campbell <campbell+netbsd-tech-pkg%mumble.net@localhost> writes:

> I think this might not be workable on netbsd-9 (and netbsd-8) because
> of netbsd-9's Heimdal sqlite3 mistake.  But, we could limit the
> builtin detection to 9 (and 8).  Users who want newer sqlite3 would
> have to PREFER_PKGSRC+=heimdal.

I'm not really comfortable with that as it makes those platforms sort of
broken (which they are now, I know).  We end up with trouble already
because some packages force newer anyway.

So I'd lean to just "make all packages reject netbsd base
implementations if that would bring in base sqlite".   Which is sort of
like PREFER_PKGSRC+=heimdal, but a bit stronger.

>> Are there other examples?  Or is this discussion truly about NetBSD 9
>> heimdal?
>
> I think it is really limited to netbsd-8/9 Heimdal.
>
>> Checking NetBSD 10, I find sqlite3 linked to
>>   /usr/lib/libhdb.so
>>   /usr/lib/libkadm5clnt.so
>>   /usr/lib/libkadm5srv.so
>>   /usr/lib/libkdc.so
>
> Does anything in pkgsrc link against them?  I doubt it -- these are
> generally only used by admin programs like kdc and kadmin/kadmind, not
> by client and server programs the way libkrb5.so and libgssapi.so are
> used extensively by anything that supports GSSAPI authentication.
>
> I grepped through
>
> https://ftp.NetBSD.org/pub/pkgsrc/packages/NetBSD/*/*.0/All/pkg_summary.gz
>
> for REQUIRES lines matching them, and the only ones I found were in
> the powerpc heimdal-7.8.0nb7 package (which is undoubtedly broken).
>
> So I don't think they are used.

That makes sense about the other libs.

> (In principle this might break if anything from pkgsrc links against
> any other base libraries that link against base libkrb5 or libgssapi
> -- but there might not be anything so we might be OK: nothing seems to
> use libbind9, libdns, libirs, libisc, libisccc, libisccfg, libns, or
> libsaslc from base; see below how I tested like for libkrb5 and
> libgssapi.)

I would say if we find that this happens, we just mark them as not
suitable for builtin.