Skip to content

gh-119048: Use shutil._use_fd_functions in test_rmtree_uses_safe_fd_v… - #157912

Open
vasiliyk wants to merge 1 commit into
python:mainfrom
vasiliyk:gh-119048-shutil
Open

vasiliyk wants to merge 1 commit into
python:mainfrom
vasiliyk:gh-119048-shutil

Conversation

@vasiliyk

@vasiliyk vasiliyk commented Sep 21, 2026

Copy link
Copy Markdown
Contributor

The test recomputed the fd-functions flag with os.listdir in os.supports_fd, but shutil switched its own check to os.scandir, so the copy in the test went stale.

Use shutil._use_fd_functions directly instead of duplicating as suggested by @encukou in the issue.

@vasiliyk
vasiliyk requested a review from giampaolo as a code owner September 21, 2026 13:45
@bedevere-app bedevere-app Bot added the tests Tests in the Lib/test dir label Sep 21, 2026
@bedevere-app

bedevere-app Bot commented Sep 21, 2026

Copy link
Copy Markdown

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

Comment thread Misc/NEWS.d/next/Tests/2026-09-21-09-48-03.gh-issue-119048.HxrBfO.rst Outdated
@bedevere-app

bedevere-app Bot commented Sep 21, 2026

Copy link
Copy Markdown

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

@vasiliyk
vasiliyk force-pushed the gh-119048-shutil branch 2 times, most recently from 64d168b to ddd72aa Compare September 21, 2026 17:21
@bedevere-app

bedevere-app Bot commented Sep 21, 2026

Copy link
Copy Markdown

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting review tests Tests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants