Skip to content

Add tests for capitalization change at the same time - #1081

Closed
RyanZim wants to merge 2 commits into
masterfrom
ryan/afonsojanu-fix/move-unicode-normalization-paths/tests
Closed

RyanZim wants to merge 2 commits into
masterfrom
ryan/afonsojanu-fix/move-unicode-normalization-paths/tests

Conversation

@RyanZim

@RyanZim RyanZim commented Sep 17, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

afonsojanu and others added 2 commits September 6, 2026 01:14
Moving a file to a name that differs only in Unicode normalization
form (NFC vs NFD) throws "Source and destination must not be the
same." on filesystems that auto-normalize, such as APFS, even though
the source and destination are visually identical and the user is
just renaming the file. This mirrors the case-insensitive rename bug
fixed in #801, so it gets the same fix: when the basenames differ but
resolve to the same inode, and they're equal after either
lowercasing or NFC normalization, treat it as a rename instead of
raising the same-path error.

Fixes #859
@RyanZim

RyanZim commented Sep 17, 2026

Copy link
Copy Markdown
Collaborator Author

Changes pushed to #1079

@RyanZim RyanZim closed this Sep 17, 2026
@RyanZim
RyanZim deleted the ryan/afonsojanu-fix/move-unicode-normalization-paths/tests branch September 17, 2026 15:30
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.

2 participants