Skip to content

Switch Microsoft.Data.Sqlite and EFCore.Sqlite to SQLite3MC.PCLRaw.bundle#38402

Open
Copilot wants to merge 3 commits into
mainfrom
copilot/change-sqlite-dependency
Open

Switch Microsoft.Data.Sqlite and EFCore.Sqlite to SQLite3MC.PCLRaw.bundle#38402
Copilot wants to merge 3 commits into
mainfrom
copilot/change-sqlite-dependency

Conversation

Copilot AI commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Changes the Microsoft.Data.Sqlite and Microsoft.EntityFrameworkCore.Sqlite packages to bundle SQLite3MC.PCLRaw.bundle instead of SQLitePCLRaw.bundle_e_sqlite3, giving shipped consumers a native SQLite build with encryption (SQLite3 Multiple Ciphers) by default, and updates the SQLite benchmarks to use the same bundle.

Changes

  • Product dependenciessrc/Microsoft.Data.Sqlite/Microsoft.Data.Sqlite.csproj and src/EFCore.Sqlite/EFCore.Sqlite.csproj now reference SQLite3MC.PCLRaw.bundle.
  • Benchmark dependencybenchmark/EFCore.Sqlite.Benchmarks/EFCore.Sqlite.Benchmarks.csproj now references SQLite3MC.PCLRaw.bundle.
  • Package version plumbing — moved the SQLite3MC.PCLRaw.bundle PackageVersion from test/Directory.Packages.props up to the root Directory.Packages.props so non-test product projects resolve it, removed the duplicate test entry, and updated $(SQLite3MCPCLRawBundleVersion) to 2.3.5 in eng/Versions.props.

Test coverage

No structural test change required — the existing Microsoft.Data.Sqlite provider matrix still exercises both bundles via separate projects and DefineConstants:

  • Microsoft.Data.Sqlite.TestsE_SQLITE3 / SQLitePCLRaw.bundle_e_sqlite3
  • Microsoft.Data.Sqlite.sqlite3mc.TestsSQLITE3MC / SQLite3MC.PCLRaw.bundle

Targeted validation also restores and builds the updated EFCore.Sqlite and EFCore.Sqlite.Benchmarks projects against the new bundle.

Notes

  • This change updates the shipped Microsoft.Data.Sqlite and Microsoft.EntityFrameworkCore.Sqlite packages, plus the SQLite benchmark project, to use SQLite3MC by default.
  • Several test-only projects intentionally continue to reference bundle_e_sqlite3 to preserve existing provider coverage.

Co-authored-by: AndriySvyryd <6539701+AndriySvyryd@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the Microsoft.Data.Sqlite meta-package to depend on the SQLite3 Multiple Ciphers (SQLite3MC) bundle by default, and moves the corresponding central package version to the repo root so the product project can resolve it.

Changes:

  • Switched src/Microsoft.Data.Sqlite/Microsoft.Data.Sqlite.csproj from SQLitePCLRaw.bundle_e_sqlite3 to SQLite3MC.PCLRaw.bundle.
  • Moved the central PackageVersion for SQLite3MC.PCLRaw.bundle from test/Directory.Packages.props to the root Directory.Packages.props and removed the duplicate test-only entry.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
test/Directory.Packages.props Removes the test-local central version entry now that the version is defined at the repo root.
src/Microsoft.Data.Sqlite/Microsoft.Data.Sqlite.csproj Changes the meta-package dependency to SQLite3MC.PCLRaw.bundle so consumers get SQLite3MC by default.
Directory.Packages.props Adds the central version entry for SQLite3MC.PCLRaw.bundle to support product builds.

@AndriySvyryd AndriySvyryd marked this pull request as ready for review June 10, 2026 21:32
@AndriySvyryd AndriySvyryd requested a review from a team as a code owner June 10, 2026 21:32
@AndriySvyryd AndriySvyryd requested a review from SamMonoRT June 10, 2026 21:33
@AndriySvyryd

Copy link
Copy Markdown
Member

@artl93 @jeffhandley @ViveliDuCh FYI

Comment thread Directory.Packages.props
Co-authored-by: AndriySvyryd <6539701+AndriySvyryd@users.noreply.github.com>
Copilot AI requested a review from wtgodbe as a code owner June 12, 2026 01:35
Copilot AI requested a review from AndriySvyryd June 12, 2026 01:35
Comment thread src/Microsoft.Data.Sqlite/Microsoft.Data.Sqlite.csproj
Co-authored-by: AndriySvyryd <6539701+AndriySvyryd@users.noreply.github.com>
Copilot AI requested review from Copilot and removed request for Copilot June 12, 2026 01:42
Copilot AI changed the title Switch Microsoft.Data.Sqlite to SQLite3MC.PCLRaw.bundle Switch Microsoft.Data.Sqlite and EFCore.Sqlite to SQLite3MC.PCLRaw.bundle Jun 12, 2026
Copilot AI requested a review from AndriySvyryd June 12, 2026 01:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants