Skip to content

Stop testing Python 3.15#51

Merged
icemac merged 1 commit into
masterfrom
config-with-c-code-template-24cb1653
Nov 27, 2025
Merged

Stop testing Python 3.15#51
icemac merged 1 commit into
masterfrom
config-with-c-code-template-24cb1653

Conversation

@icemac
Copy link
Copy Markdown
Member

@icemac icemac commented Nov 27, 2025

It is broken probably because we need a fix in cffi, see https://github.com/zopefoundation/Persistence/actions/runs/19610857054/job/56156444668

@icemac icemac requested a review from dataflake November 27, 2025 08:13
@icemac icemac self-assigned this Nov 27, 2025
@icemac icemac enabled auto-merge (squash) November 27, 2025 08:13
@icemac icemac disabled auto-merge November 27, 2025 08:13
@icemac icemac enabled auto-merge (squash) November 27, 2025 08:14
Copy link
Copy Markdown
Member

@dataflake dataflake left a comment

Choose a reason for hiding this comment

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

At the root of the recurring CFFI issues is the CFFI dependency in the persistent package. I wonder why in the world that is even there.

@icemac icemac merged commit af55692 into master Nov 27, 2025
48 checks passed
@icemac icemac deleted the config-with-c-code-template-24cb1653 branch November 27, 2025 08:17
@davisagli
Copy link
Copy Markdown
Member

@dataflake It looks like it is used for the pure-Python implementation of the pickle cache, and was an optional dependency until zopefoundation/persistent#107. Maybe it's worth re-adding the other implementation that was removed there and making cffi optional again, since we have this recurring problem where there aren't wheels available for cffi on new Python versions as early as we'd like.

@dataflake
Copy link
Copy Markdown
Member

@davisagli Thanks for the hint. I added a comment in zopefoundation/persistent#107 because I am just tired of the constant issues. The reasons mentioned for the change, like code simplification and test coverage in persistent, are not worth this never ending hassle.

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