Add compile-time options warning to SQLite spatial docs#5320
Merged
AndriySvyryd merged 3 commits intomainfrom Apr 2, 2026
Merged
Add compile-time options warning to SQLite spatial docs#5320AndriySvyryd merged 3 commits intomainfrom
AndriySvyryd merged 3 commits intomainfrom
Conversation
Agent-Logs-Url: https://github.com/dotnet/EntityFramework.Docs/sessions/0cbb646c-c532-479e-92b1-566ebc8175da Co-authored-by: AndriySvyryd <6539701+AndriySvyryd@users.noreply.github.com>
Agent-Logs-Url: https://github.com/dotnet/EntityFramework.Docs/sessions/abc1b896-bfcb-4a24-9790-c9c8f88e222f Co-authored-by: AndriySvyryd <6539701+AndriySvyryd@users.noreply.github.com>
Copilot created this pull request from a session on behalf of
AndriySvyryd
April 2, 2026 18:47
View session
There was a problem hiding this comment.
Pull request overview
Adds guidance to the EF Core SQLite spatial documentation to warn that switching from the bundled SQLite to a system-installed SQLite (for SpatiaLite compatibility) can change available SQLite features due to differing compile-time options.
Changes:
- Extends the existing
[!IMPORTANT]guidance to mention compile-time option differences between bundled vs system SQLite. - Calls out potential feature impacts (e.g., math functions, full-text search) and advises verification.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
roji
approved these changes
Apr 2, 2026
|
|
||
| > [!IMPORTANT] | ||
| > Don't use `Microsoft.EntityFrameworkCore.Sqlite` or `Microsoft.Data.Sqlite` with SpatiaLite on macOS and Linux. Both packages pull in `SQLitePCLRaw.bundle_e_sqlite3` by default—a bundled version of SQLite that is incompatible with system-installed Sqlite. Using it may result in a silent crash at run time. Use `Microsoft.EntityFrameworkCore.Sqlite.Core` or `Microsoft.Data.Sqlite.Core` instead, together with the system SQLite provider as shown below. | ||
| > Don't use `Microsoft.EntityFrameworkCore.Sqlite` or `Microsoft.Data.Sqlite` with SpatiaLite on macOS and Linux. Both packages pull in `SQLitePCLRaw.bundle_e_sqlite3` by default—a bundled version of SQLite that is incompatible with system-installed SQLite. Using it may result in a silent crash at run time. Use `Microsoft.EntityFrameworkCore.Sqlite.Core` or `Microsoft.Data.Sqlite.Core` instead, together with the system SQLite provider as shown below. |
Member
There was a problem hiding this comment.
Still seems overly aggressive - IIUC the problem is only with using these packages when SpatiaLite is machine-installed on macOS/Linux, which I'd suspect isn't necessarily the normal case.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
When switching from the bundled SQLite (
SQLitePCLRaw.bundle_e_sqlite3) to a system-installed SQLite for SpatiaLite compatibility, the available compile-time options may differ—silently breaking features like math functions and full-text search.[!IMPORTANT]block inentity-framework/core/providers/sqlite/spatial.mdwarning users to verify feature availability against their system SQLite build