Skip to content

Conversation

@devonh
Copy link
Member

@devonh devonh commented Apr 16, 2025

Reverts #18260

It is causing a failure when building release debs for debian:bullseye with the following error:

sqlite3.OperationalError: near "RETURNING": syntax error

@devonh devonh requested a review from a team as a code owner April 16, 2025 16:04
@github-actions github-actions bot deployed to PR Documentation Preview April 16, 2025 16:05 Active
Copy link
Member

@anoadragon453 anoadragon453 left a comment

Choose a reason for hiding this comment

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

@devonh could you add a quick newsfile to get CI to run/pass? We can then remove it before merging.

@github-actions github-actions bot deployed to PR Documentation Preview April 16, 2025 16:11 Active
@devonh devonh merged commit 89cb613 into develop Apr 16, 2025
9 of 10 checks passed
@devonh devonh deleted the revert-18260-anoa/export_total_message_count branch April 16, 2025 16:41
@github-actions github-actions bot deployed to PR Documentation Preview April 16, 2025 16:42 Active
MadLittleMods added a commit that referenced this pull request Apr 30, 2025
This was the original reason the PR was reverted, see
#18346

> The `RETURNING` syntax has been supported by SQLite since version 3.35.0 (2021-03-12).
>
> *-- https://www.sqlite.org/lang_returning.html*

Synapse supports...

> The oldest supported version of SQLite is the version [provided](https://packages.debian.org/bullseye/libsqlite3-0) by [Debian oldstable](https://wiki.debian.org/DebianOldStable).
>
> *-- https://element-hq.github.io/synapse/latest/deprecation_policy.html*

which currently is https://packages.debian.org/bullseye/sqlite3 -> `3.34.1-3+deb11u1`

We have `self.db_pool.engine.supports_returning` to detect whether we can use `RETURNING`.
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