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

GH-104947: Make pathlib.PureWindowsPath comparisons consistent across platforms by barneygale · Pull Request #104948 · python/cpython · GitHub

/ cpython Public

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 commented May 25, 2023
edited
Loading

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.
barneygale changed the title [WIP] GH-104947: Make pathlib.PureWindowsPath comparisons consistent across platforms GH-104947: Make pathlib.PureWindowsPath comparisons consistent across platforms May 25, 2023
barneygale marked this pull request as ready for review May 25, 2023 22:44
barneygale removed the tests Tests in the Lib/test dir label May 25, 2023
barneygale enabled auto-merge (squash) May 26, 2023 17:42
barneygale merged commit ad0be36 into python:main May 26, 2023

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>

Copy link
Copy Markdown

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

bedevere-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>
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.

4 participants


Back | FazBrowse Home | New Git URL