Skip to content

[3.10] gh-123693: Use platform-agnostic semantics when processing zipfile.Path.name.#123694

Merged
ambv merged 10 commits into
python:3.10from
jaraco:gh-123693/platform-agnostic-name
Sep 5, 2024
Merged

[3.10] gh-123693: Use platform-agnostic semantics when processing zipfile.Path.name.#123694
ambv merged 10 commits into
python:3.10from
jaraco:gh-123693/platform-agnostic-name

Conversation

@jaraco

@jaraco jaraco commented Sep 4, 2024

Copy link
Copy Markdown
Member

Depends on, merge after GH-123426.

Closes #123693.

@jaraco

jaraco commented Sep 4, 2024

Copy link
Copy Markdown
Member Author

I chose not to replace the patch with exactly the patch that is found in Python 3.11 and later, because that changed other semantics. I'm expecting this selective change to allow the tests to pass and address the missed expectation.

@jaraco

jaraco commented Sep 4, 2024

Copy link
Copy Markdown
Member Author

Until this PR is rebased onto main after merging the dependent PR, here is the diff for this PR.

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

LGTM

@ambv

ambv commented Sep 4, 2024

Copy link
Copy Markdown
Contributor

You can rebase now.

@jaraco

jaraco commented Sep 4, 2024

Copy link
Copy Markdown
Member Author

I've merged and the diff is now good. The macOS failure is unrelated. I did not rebase, as the squash merge will handle that. Please merge whenever.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants