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

[3.13] gh-123880: Allow recursive import of single-phase-init modules (GH-123950) #124273

Merged
merged 1 commit into from
Sep 23, 2024

Conversation

miss-islington
Copy link
Contributor

@miss-islington miss-islington commented Sep 20, 2024

(cherry picked from commit aee219f)

Co-authored-by: Petr Viktorin encukou@gmail.com
Co-authored-by: Shantanu 12621235+hauntsaninja@users.noreply.github.com
Co-authored-by: Brett Cannon brett@python.org

…ythonGH-123950)

(cherry picked from commit aee219f)

Co-authored-by: Petr Viktorin <encukou@gmail.com>
Co-authored-by: Shantanu <12621235+hauntsaninja@users.noreply.github.com>
Co-authored-by: Brett Cannon <brett@python.org>
@bedevere-bot
Copy link

🤖 New build scheduled with the buildbot fleet by @encukou for commit 58ba5ba 🤖

If you want to schedule another build, you need to add the 🔨 test-with-buildbots label again.

@bedevere-bot bedevere-bot removed the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Sep 20, 2024
@Yhg1s
Copy link
Member

Yhg1s commented Sep 23, 2024

FYI, pushing this to 3.13.1.

@hauntsaninja
Copy link
Contributor

hauntsaninja commented Sep 23, 2024

@Yhg1s good to know, is that at all open for discussion? This means that most mypyc compiled projects (like mypy and black and charset_normalizer) will be broken on 3.13.0 when they ship 3.13 mypyc-compiled wheels.

@Yhg1s
Copy link
Member

Yhg1s commented Sep 23, 2024

Sure, it's open for discussion. Given that this showed up all the way at the tail end of the release candidate phase, I'm assuming it must not be that pressing (or it would've shown up before :). It sounds like it is pressing, and the actual change is not that complicated (although slightly more complicated than I would've liked between the release candidate and the final release).

Why didn't this show up before? How are we going to make similar issues show up earlier in the future?

@hauntsaninja
Copy link
Contributor

hauntsaninja commented Sep 23, 2024

It didn't show up before primarily because there were other issues running mypyc with 3.13 that showed up before this one. It looks like there was also a period of 3 weeks where it was known but not acted on (following #121489 (comment) ) due to a combination of classic open source volunteer time and communication issues + also I think there was some misplaced suspicion that the issue was something specific to mypyc's test suite.

Timeline on mypy's side is something like:

3.12 was easier, I think we had things green before first release candidate.

Some things that could help:

  • I think if Github offered a true nightly build of Python that would help. There was probably a bit of "oh well we're waiting for upstream"
  • mypy could probably have started testing during alphas
  • I'd planned to make sure things were greenable in rc1, but had some health issues that prevented me from doing so

@Yhg1s
Copy link
Member

Yhg1s commented Sep 23, 2024

FWIW, we discussed this at the sprint as well, and it's clear that this should go into 3.13.0. We should try not to break mypyc (and similar users) quite so much, to make it easier for mypyc to find these issues for us in the future :)

@Yhg1s Yhg1s merged commit de8dc92 into python:3.13 Sep 23, 2024
111 of 114 checks passed
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.

5 participants