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

Enable importlib.metadata backend on Python 3.11 #11044

Merged
merged 1 commit into from May 10, 2022

Conversation

uranusjr
Copy link
Member

With 3.11 beta coming out next month, we can get this ready for testing.

Similar to the sysconfig transition, this builds in some hidden switches for downstream distributors to use, in case they need it. (see #10647)

@uranusjr uranusjr force-pushed the importlib-metadata-backend-in-3.11 branch from a1986d9 to c07ae74 Compare April 19, 2022 09:15
@@ -219,11 +219,12 @@ jobs:
env:
TEMP: "R:\\Temp"

# TODO: Remove this when we add Python 3.11 to CI.
Copy link
Member

Choose a reason for hiding this comment

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

We can do this now FWIW, 3.11.0-alpha - 3.11?

Copy link
Member Author

Choose a reason for hiding this comment

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

Let me try.

Copy link
Member Author

Choose a reason for hiding this comment

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

Ah, it’s a bit difficult to convert 3.11.0-alpha.7 to 3.11 for nox -s test-{{ python }}

Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Member Author

Choose a reason for hiding this comment

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

Ah, this is the syntax I was looking for, thanks

${{ matrix.nox-python || matrix.python }}

Copy link
Member Author

Choose a reason for hiding this comment

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

Seem like it’s not ready for virtualenv (which we need for Nox) unfortunately

ERROR: The executable /home/runner/work/pip/pip/.nox/test-3-11/bin/python3.11 is not functioning
ERROR: It thinks sys.prefix is '/opt/hostedtoolcache/Python/3.11.0-alpha.7/x64' (should be '/home/runner/work/pip/pip/.nox/test-3-11')
ERROR: virtualenv is not compatible with this system or executable
Running virtualenv with interpreter /opt/hostedtoolcache/Python/3.11.0-alpha.7/x64/bin/python3.11

Copy link
Member

Choose a reason for hiding this comment

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

Yea, we're stuck on virtualenv 16.x for our test suite. That's now a proper problem. :)

news/10709.process.rst Outdated Show resolved Hide resolved
@uranusjr uranusjr force-pushed the importlib-metadata-backend-in-3.11 branch 4 times, most recently from a176c5e to c07ae74 Compare April 20, 2022 00:11
@github-actions github-actions bot added the needs rebase or merge PR has conflicts with current master label May 3, 2022
@pradyunsg
Copy link
Member

@uranusjr Let's rebase this and get this released this week, before the Python 3.11 betas begin?

@uranusjr uranusjr force-pushed the importlib-metadata-backend-in-3.11 branch 2 times, most recently from b0f754f to 930d68e Compare May 3, 2022 18:12
@pypa-bot pypa-bot removed the needs rebase or merge PR has conflicts with current master label May 3, 2022
@uranusjr uranusjr added this to the 22.1 milestone May 3, 2022
@pradyunsg
Copy link
Member

pradyunsg commented May 9, 2022

So... it looks like we didn't get this into beta1. Are we comfortable making a change to how Python 3.11+ behaves in a pip release immediately after the beta period has started?

@pradyunsg
Copy link
Member

pradyunsg commented May 10, 2022

@uranusjr nudge nudge. This is currently blocking 22.1.

@uranusjr
Copy link
Member Author

Sorry I’m attending another conference this week; might get some time to get this in later today.

@pypa pypa deleted a comment from vuka1997 May 10, 2022
@uranusjr uranusjr force-pushed the importlib-metadata-backend-in-3.11 branch from 930d68e to bd9bcef Compare May 10, 2022 17:35
@uranusjr uranusjr requested a review from pradyunsg May 10, 2022 17:36
@uranusjr uranusjr merged commit c86f9f1 into pypa:main May 10, 2022
@uranusjr uranusjr deleted the importlib-metadata-backend-in-3.11 branch May 10, 2022 18:34
@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 26, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants