Skip to content

GH-104947: Make pathlib.PureWindowsPath comparisons consistent across platforms - #104948

Merged
barneygale merged 3 commits into
python:mainfrom
barneygale:gh-104947-pathlib-use-str-lower
May 26, 2023
Merged

GH-104947: Make pathlib.PureWindowsPath comparisons consistent across platforms#104948
barneygale merged 3 commits into
python:mainfrom
barneygale:gh-104947-pathlib-use-str-lower

Conversation

@barneygale

@barneygalebarneygale commented May 25, 2023

Copy link
Copy Markdown
Contributor

Use str.lower() rather than ntpath.normcase() to normalize case of Windows paths. This restores behaviour from Python 3.11.

…across OSs
Use `str.lower()` rather than `ntpath.normcase()` to normalize case of
Windows paths. This restores behaviour from Python 3.11.
@barneygalebarneygale changed the title [WIP] GH-104947: Make pathlib.PureWindowsPath comparisons consistent across platformsGH-104947: Make pathlib.PureWindowsPath comparisons consistent across platformsMay 25, 2023
@barneygale
barneygale marked this pull request as ready for review May 25, 2023 22:44
@barneygalebarneygale removed the tests Tests in the Lib/test dir label May 25, 2023
@barneygale
barneygale enabled auto-merge (squash) May 26, 2023 17:42
@barneygale
barneygale merged commit ad0be36 into python:mainMay 26, 2023
@miss-islington

Copy link
Copy Markdown
Contributor

Thanks @barneygale for the PR 🌮🎉.. I'm working now to backport this PR to: 3.12.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request May 26, 2023
…across platforms (pythonGH-104948)
Use `str.lower()` rather than `ntpath.normcase()` to normalize case of
Windows paths. This restores behaviour from Python 3.11.
(cherry picked from commit ad0be36)
Co-authored-by: Barney Gale <barney.gale@gmail.com>
Co-authored-by: Gregory P. Smith <greg@krypto.org>
@bedevere-bot

Copy link
Copy Markdown

GH-104990 is a backport of this pull request to the 3.12 branch.

@bedevere-botbedevere-bot removed the needs backport to 3.12 only security fixes label May 26, 2023
barneygale added a commit that referenced this pull request May 26, 2023
… across platforms (GH-104948) (GH-104990)
Use `str.lower()` rather than `ntpath.normcase()` to normalize case of
Windows paths. This restores behaviour from Python 3.11.
(cherry picked from commit ad0be36)
Co-authored-by: Barney Gale <barney.gale@gmail.com>
Co-authored-by: Gregory P. Smith <greg@krypto.org>
Sign up for freeto 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

@barneygale@miss-islington@bedevere-bot@gpshead