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

Add patch for use with Python 3.11 #34

Merged
merged 2 commits into from
Oct 25, 2022

Conversation

chrisburr
Copy link
Member

Fixing the issue seen in conda-forge/pytest-feedstock#152

@conda-forge-linter
Copy link

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@chrisburr
Copy link
Member Author

There's something weird still going on as it's not looking in lib/python3.11/lib-dynload meaning modules like math can't be imported...

@chrisburr
Copy link
Member Author

My current best guess is that python/cpython#29041 is the change that has broken crossenv.

@chrisburr
Copy link
Member Author

Another candidate the frozen site module: python/cpython#89183

Though defrosting it doesn't fix the issue:

$ ....../bin/python3.11 -X frozen_modules=off -c 'import site; print(site)'
Fatal Python error: init_import_site: Failed to import the site module
Python runtime state: initialized
Traceback (most recent call last):
  File "/home/cburr/miniconda3/conda-bld/pytest_1666698751133/_build_env/venv/lib/site.py", line 22, in <module>
    import importlib.abc
  File "/home/cburr/miniconda3/conda-bld/pytest_1666698751133/_build_env/lib/python3.11/importlib/abc.py", line 19, in <module>
    from .resources.abc import ResourceReader, Traversable, TraversableResources
  File "/home/cburr/miniconda3/conda-bld/pytest_1666698751133/_build_env/lib/python3.11/importlib/resources/__init__.py", line 3, in <module>
    from ._common import (
  File "/home/cburr/miniconda3/conda-bld/pytest_1666698751133/_build_env/lib/python3.11/importlib/resources/_common.py", line 3, in <module>
    import tempfile
  File "/home/cburr/miniconda3/conda-bld/pytest_1666698751133/_build_env/lib/python3.11/tempfile.py", line 45, in <module>
    from random import Random as _Random
  File "/home/cburr/miniconda3/conda-bld/pytest_1666698751133/_build_env/lib/python3.11/random.py", line 49, in <module>
    from math import log as _log, exp as _exp, pi as _pi, e as _e, ceil as _ceil
ModuleNotFoundError: No module named 'math'

@chrisburr
Copy link
Member Author

See conda-forge/cross-python-feedstock#57 for the solution.

@isuruf
Copy link
Member

isuruf commented Oct 25, 2022

@chrisburr, please wait for the maintainers to review PRs in the future. Can you add yourself as a maintainer to all the feedstocks you merged today? crossenv, cross-python, python_abi, python etc?

@chrisburr
Copy link
Member Author

@isuruf sorry about that, I should have been more patient waiting with these.

Can you add yourself as a maintainer to all the feedstocks you merged today? crossenv, cross-python, python_abi, python etc?

Will do 👍

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