FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

[3.12] gh-125783: Add tests to prevent regressions with the combination of `ctypes` and metaclasses. (GH-125881) by miss-islington · Pull Request #125988 · python/cpython · GitHub

/ cpython Public

[3.12] gh-125783: Add tests to prevent regressions with the combination of ctypes and metaclasses. (GH-125881) - #125988

Merged
encukou merged 2 commits into
python:3.12from
miss-islington:backport-1384409-3.12
Oct 29, 2024
Merged

[3.12] gh-125783: Add tests to prevent regressions with the combination of ctypes and metaclasses. (GH-125881)#125988
encukou merged 2 commits into
python:3.12from
miss-islington:backport-1384409-3.12

Conversation

miss-islington commented Oct 25, 2024
edited by bedevere-app Bot
Loading

Copy link
Copy Markdown
Contributor

…n of `ctypes` and metaclasses. (pythonGH-125881)

(cherry picked from commit 1384409)

Co-authored-by: Jun Komoda <45822440+junkmd@users.noreply.github.com>

junkmd commented Oct 26, 2024
edited
Loading

Copy link
Copy Markdown
Contributor

Unlike main and 3.13, 3.12 does not yet include the test_ctypes._support module, which is causing test failures.

I think a maintainer with the appropriate permissions needs to do such as git checkout 1384409 -- Lib/test/test_ctypes/_support.py on this branch.

encukou commented Oct 28, 2024

Copy link
Copy Markdown
Member

I'll try backporting all of #113727.

This partially backports be89ee5
(python#113727)

Co-authored-by: Erlend E. Aasland <erlend.aasland@protonmail.com>

encukou commented Oct 28, 2024

Copy link
Copy Markdown
Member

Nope, too invasive. I'll add _support.py as suggested. If we backport the whole commit later, _support.py shouldn't conflict with it.

encukou merged commit bce9df9 into python:3.12 Oct 29, 2024
miss-islington deleted the backport-1384409-3.12 branch January 2, 2026 18:00
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip news tests Tests in the Lib/test dir topic-ctypes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants


Back | FazBrowse Home | New Git URL