Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support py310 & py311 #2088

Merged
merged 8 commits into from
Sep 6, 2023
Merged

Support py310 & py311 #2088

merged 8 commits into from
Sep 6, 2023

Conversation

kclowes
Copy link
Collaborator

@kclowes kclowes commented Dec 9, 2022

What was wrong?

  • pyethash updates for Python 3.10 led to segfaults. PoW, along with the Ethash algorithm, was de-prioritized in Remove dependence on ethash (pyethash) #2121 which opened the way for a much cleaner version of this PR to add support to python 3.10 and 3.11.

How was it fixed?

  • Add support for Python 3.10 and 3.11
  • Remove LevelDB support via plyvel as that library is no longer maintained and lacked certain system binaries

Todo:

  • Clean up commit history

Cute Animal Picture

put a cute animal picture link inside the parentheses

@fselmo fselmo force-pushed the support-py310 branch 15 times, most recently from f961634 to 7bdfa9b Compare February 9, 2023 22:36
@fselmo fselmo force-pushed the support-py310 branch 5 times, most recently from ec0c61d to 08e1d27 Compare July 26, 2023 15:36
@fselmo fselmo force-pushed the support-py310 branch 5 times, most recently from 71175cf to e1edece Compare September 6, 2023 18:27
kclowes and others added 2 commits September 6, 2023 13:51
- Add py310 runs to circleci config and tox
- Use current master branch of ``pyethash`` as version to test broken tests
- Update ``tox`` to ``>=4.0.0`` + related changes (e.g. ``whitelist_externals`` -> ``allowlist_externals``)
- Remove ``python_paths`` from ``pytest`` config
- Update ``pytest-timeout`` to ``>=2.0.0<3``
- Update ``blake2b-py``
- Update ``pytest-xdist``
@fselmo fselmo changed the title Support py310 Support py310 & py311 Sep 6, 2023
fselmo added a commit to kclowes/py-evm that referenced this pull request Sep 6, 2023
@fselmo fselmo marked this pull request as ready for review September 6, 2023 20:02
Copy link
Collaborator

@pacrob pacrob left a comment

Choose a reason for hiding this comment

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

lgtm!

- Can try to turn ``warn_unused_ignores`` back on after py39 support is dropped
It seems like binaries for some platforms are hard to come by for plyvel. It may no longer be maintained. We either subscribe to plyvel-ci or something similar or rip leveldb support out. I think at some point this may have served a more specific purpose anyhow for the Trinity client and perhaps there is little use for what seems like a decent amount of maintenance.
@fselmo fselmo merged commit 8d53346 into ethereum:master Sep 6, 2023
39 of 44 checks passed
@fselmo fselmo mentioned this pull request Sep 13, 2023
fselmo added a commit to fselmo/py-evm that referenced this pull request Oct 9, 2023
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