conda-forge / sqlite-feedstock

A conda-smithy repository for sqlite.
BSD 3-Clause "New" or "Revised" License
2 stars 38 forks source link

Ensure URI support is always on #67

Open tschoonj opened 3 years ago

tschoonj commented 3 years ago

Checklist

This PR introduces compile-time builtin support for URIs, removing the need to enable support for them at run-time, which cannot always be easily accomplished when using sqlite bindings (R!).

The SQLite docs contain a section about backwards compatibility regarding this option, which I don't think will impact any users.

tschoonj commented 3 years ago

@conda-forge/sqlite would it be possible to review this PR? Thanks in advance!!

jrandall commented 3 years ago

See upstream documentation at https://sqlite.org/uri.html for information about the feature that this PR would enable.

It may be relevant to reviewers to note that the SQLite developers state that it is safe for most applications to enable URI processing:

Since SQLite always interprets any filename that does not begin with "file:" as an ordinary filename regardless of the URI setting, and because it is very unusual to have an actual file begin with "file:", it is safe for most applications to enable URI processing even if URI filenames are not currently being used.