Skip to content

Conversation

@barneygale
Copy link
Contributor

@barneygalebarneygale commented May 25, 2023

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
@barneygalebarneygale 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
@barneygalebarneygale enabled auto-merge (squash) May 26, 2023 17:42
@barneygalebarneygale merged commit ad0be36 into python:mainMay 26, 2023
@miss-islington
Copy link
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

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