Created attachment 233119 [details] Update to 1.56.0 - Update to 1.56.0 - Note important changes since 1.55.2 in pkg-message.in Following is an excerpt from test suite: tests.replication.slave.storage.test_events.SlavedEventStoreTestCase.test_redactions tests.replication.slave.storage.test_receipts.SlavedReceiptTestCase.test_receipt tests.replication.tcp.streams.test_account_data.AccountDataStreamTestCase.test_update_function_global_account_data_limit tests.replication.tcp.streams.test_account_data.AccountDataStreamTestCase.test_update_function_room_account_data_limit tests.replication.tcp.streams.test_events.EventsStreamTestCase.test_backwards_stream_id tests.replication.tcp.streams.test_events.EventsStreamTestCase.test_update_function_event_row_limit tests.replication.tcp.streams.test_events.EventsStreamTestCase.test_update_function_huge_state_change tests.replication.tcp.streams.test_events.EventsStreamTestCase.test_update_function_state_row_limit tests.replication.tcp.streams.test_federation.FederationStreamTestCase.test_catchup tests.replication.tcp.streams.test_receipts.ReceiptsStreamTestCase.test_receipt tests.replication.tcp.streams.test_typing.TypingStreamTestCase.test_reset tests.replication.tcp.streams.test_typing.TypingStreamTestCase.test_typing ------------------------------------------------------------------------------- Ran 2278 tests in 1216.386s PASSED (skips=40, successes=2238)
Created attachment 233156 [details] Update to 1.56.0 Sorry about the previous patch.
Patch looks good, but misses two minor dependency changes: * We now have a dependency on jinja2>=3.0.0. * We have a version cap on signedjson (<=1.1.1). The latter is only necessary for typechecking during developement and thus doesn't really matter to us (see [1]). I'd suggest removing that check from synapse's python_dependencies.py checking, like we do for the artificial cryptography check. Otherwise this will bite us if someone updates signedjson beyond 1.1.1. I've also poked upstream about this and they agree. :D I've updated your patch with these two minor changes applied. Still passes tests, builds fine and runs fine. :) Since this release includes changes that will cause synapse to refuse starting unless the configuration is adjusted and/or a complete (and time consuming) DB migration takes place, I think it would be worthwhile to also mention this in UPGRADING. :D Cheers, Sascha [1] https://github.com/matrix-org/synapse/issues/12324
Created attachment 233162 [details] net-im/py-matrix-synapse: Update to 1.56.0 (w/ dependency adjustments)
Created attachment 233177 [details] Update to 1.56.0 v2 It includes all the changes in the previous diffs + UPDATING entry. Thanks!
Looks good to me, let's get this merged. :)
A commit in branch main references this bug: URL: https://cgit.FreeBSD.org/ports/commit/?id=76ad2b55b71f9ba18a205d9ecdb248b78d626a0c commit 76ad2b55b71f9ba18a205d9ecdb248b78d626a0c Author: Ashish SHUKLA <ashish@FreeBSD.org> AuthorDate: 2022-04-10 11:45:30 +0000 Commit: Ashish SHUKLA <ashish@FreeBSD.org> CommitDate: 2022-04-13 13:58:07 +0000 net-im/py-matrix-synapse: Update to 1.56.0 - Mention important changes related to the upgrade to UPDATING - Update the dependency constraints Co-authored-by: Sascha Biberhofer <ports at skyforge dot at> PR: 263193 Approved by: Sascha Biberhofer <ports at skyforge dot at> (maintainer) UPDATING | 22 ++++++++++++++++++++++ net-im/py-matrix-synapse/Makefile | 4 ++-- net-im/py-matrix-synapse/distinfo | 6 +++--- .../files/patch-synapse_python__dependencies.py | 13 +++++++++++-- net-im/py-matrix-synapse/files/pkg-message.in | 17 +++++++++++++++++ 5 files changed, 55 insertions(+), 7 deletions(-)
Committed, thanks!
A commit in branch 2022Q2 references this bug: URL: https://cgit.FreeBSD.org/ports/commit/?id=9a15ef04b4cc306bf68abfcd474eb4a1f726ccf2 commit 9a15ef04b4cc306bf68abfcd474eb4a1f726ccf2 Author: Ashish SHUKLA <ashish@FreeBSD.org> AuthorDate: 2022-04-10 11:45:30 +0000 Commit: Ashish SHUKLA <ashish@FreeBSD.org> CommitDate: 2022-06-29 04:09:54 +0000 net-im/py-matrix-synapse: Update to 1.56.0 - Mention important changes related to the upgrade to UPDATING - Update the dependency constraints Co-authored-by: Sascha Biberhofer <ports at skyforge dot at> PR: 263193 Approved by: Sascha Biberhofer <ports at skyforge dot at> (maintainer) (cherry picked from commit 76ad2b55b71f9ba18a205d9ecdb248b78d626a0c) net-im/py-matrix-synapse/Makefile | 4 ++-- net-im/py-matrix-synapse/distinfo | 6 +++--- .../files/patch-synapse_python__dependencies.py | 13 +++++++++++-- net-im/py-matrix-synapse/files/pkg-message.in | 17 +++++++++++++++++ 4 files changed, 33 insertions(+), 7 deletions(-)