Skip to content

gh-133517: Remove os.listdrive, os.listvolumes and os.listmounts in non windows desktop builds#133518

Merged
zooba merged 12 commits into
python:mainfrom
maxbachmann:patch-1
May 7, 2025
Merged

gh-133517: Remove os.listdrive, os.listvolumes and os.listmounts in non windows desktop builds#133518
zooba merged 12 commits into
python:mainfrom
maxbachmann:patch-1

Conversation

@maxbachmann

@maxbachmann maxbachmann commented May 6, 2025

Copy link
Copy Markdown
Contributor

@maxbachmann maxbachmann changed the title gh-133517: GetLogicalDriveStringsW is only available on windows desktop gh-133517: Remove os.listdrive` and os.listvolumes in non windows desktop builds May 6, 2025
@maxbachmann maxbachmann changed the title gh-133517: Remove os.listdrive` and os.listvolumes in non windows desktop builds gh-133517: Remove os.listdrive, os.listvolumes and os.listmounts in non windows desktop builds May 6, 2025
@maxbachmann

Copy link
Copy Markdown
Contributor Author

This is ready for review now

Comment thread Misc/NEWS.d/next/Library/2025-05-06-14-44-55.gh-issue-133517.Ca6NgW.rst Outdated
@chris-eibl

Copy link
Copy Markdown
Member

LGTM. Pinging @zooba for final steps.

…a6NgW.rst

Co-authored-by: Chris Eibl <138194463+chris-eibl@users.noreply.github.com>
Comment thread Modules/posixmodule.c

@zooba zooba left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Will merge once the branch unlocks. This can be backported if needed

@maxbachmann

Copy link
Copy Markdown
Contributor Author

Will merge once the branch unlocks. This can be backported if needed

I don't need a backport myself. We have a patched version of Cpython anyway and just drop the patches when we upgrade to a version that includes the changes.

@zooba zooba merged commit 427f8c3 into python:main May 7, 2025
43 checks passed
@zooba

zooba commented May 7, 2025

Copy link
Copy Markdown
Member

Well, the offer of a backport is there if someone says they need it. But if not, let's not bother.

Pranjal095 pushed a commit to Pranjal095/cpython that referenced this pull request Jul 12, 2025
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.

4 participants