Skip to content

windows: Sync SQLite build flags from the UNIX build - #800

Merged
geofft merged 1 commit into
mainfrom
geofft/sqlite-windows-flags
Sep 18, 2025
Merged

windows: Sync SQLite build flags from the UNIX build#800
geofft merged 1 commit into
mainfrom
geofft/sqlite-windows-flags

Conversation

@geofft

@geofft geofft commented Sep 18, 2025

Copy link
Copy Markdown
Collaborator

Notably this adds

Notably this adds
* SQLITE_ENABLE_FTS3_PARENTHESIS (syntax change, see #550)
* SQLITE_ENABLE_DBSTAT_VTAB (#309)
* SQLITE_ENABLE_GEOPOLY (historically present on UNIX, maybe see #694)
Comment thread cpython-windows/build.py
Comment on lines +573 to +579
b"SQLITE_ENABLE_DBSTAT_VTAB",
b"SQLITE_ENABLE_FTS3",
b"SQLITE_ENABLE_FTS3_PARENTHESIS",
b"SQLITE_ENABLE_FTS4",
b"SQLITE_ENABLE_FTS5",
b"SQLITE_ENABLE_GEOPOLY",
b"SQLITE_ENABLE_RTREE",

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

It sounds like this list should live somewhere else!

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Yeah, I sort of thought it was in code somewhere already, but it's currently inline in the shell script on the UNIX side. We should maybe do something like extension-modules.yml for our native builds.

@geofft
geofft merged commit 3811863 into main Sep 18, 2025
22 checks passed
@geofft
geofft deleted the geofft/sqlite-windows-flags branch September 18, 2025 15:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants