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

[3.13] gh-157579: Fix race condition in the cleanup of `tempfile.TemporaryDirectory` (GH-157580) by miss-islington · Pull Request #158429 · python/cpython · GitHub

Repository navigation

[3.13] gh-157579: Fix race condition in the cleanup of tempfile.TemporaryDirectory (GH-157580) - #158429

Open
miss-islington wants to merge 6 commits into
python:3.13from
miss-islington:backport-5c20517-3.13
Open

miss-islington wants to merge 6 commits into
python:3.13from
miss-islington:backport-5c20517-3.13

Conversation

miss-islington commented Sep 29, 2026 •
edited by bedevere-app Bot
Loading

Copy link
Copy Markdown
Contributor

(cherry picked from commit 5c20517)

Co-authored-by: Stan Ulbrych stan@python.org
Co-authored-by: Petr Viktorin encukou@gmail.com

…raryDirectory` (pythonGH-157580)

(cherry picked from commit 5c20517)

Co-authored-by: Stan Ulbrych <stan@python.org>
Co-authored-by: Petr Viktorin <encukou@gmail.com>

read-the-docs-community Bot commented Sep 29, 2026 •
edited
Loading

Copy link
Copy Markdown

This partially backports commit 86b8617
(pythonGH-146195); making existing tests use the helper is omitted.
encukou added the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Sep 30, 2026

Copy link
Copy Markdown

🤖 New build scheduled with the buildbot fleet by @encukou for commit 8da0b0a 🤖

Results will be shown at:

https://buildbot.python.org/all/#/grid?branch=refs%2Fpull%2F158429%2Fmerge

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

bedevere-bot removed the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Sep 30, 2026

Yhg1s commented Sep 30, 2026

Copy link
Copy Markdown
Member

The Android buildbot failures, at least, seem relevant?

encukou commented Oct 9, 2026

Copy link
Copy Markdown
Member

Thanks for merging the test infrastructure!
The issue is now clear: Android's fchmodat works incorrectly before API level 23, as configure.ac in blocked_funcs.

In newer branches, we might want the test to be less or more eager -- I can find arguments for both.
But for 3.13, the correct thing to do is to skip the test.

encukou added the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Oct 9, 2026

Copy link
Copy Markdown

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

Results will be shown at:

https://buildbot.python.org/all/#/grid?branch=refs%2Fpull%2F158429%2Fmerge

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

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants


Back | FazBrowse Home | New Git URL