Skip to content

rename: Add and document errors caused by circular loops #24159

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

gooncreeper
Copy link
Contributor

@gooncreeper gooncreeper commented Jun 12, 2025

On POSIX systems, EINVAL is used to indicate circular loops;
additionally, corresponding with the use of the error in other
filesystem syscalls, it is used to indicate bad file names.

Based on empirical testing on Windows 10, SHARING_VIOLATION is
used to indicate a circular loop. On Windows 11, INVALID_PARAMATER
is used instead.

@gooncreeper gooncreeper force-pushed the rename-circular-error branch from bddeea1 to 5f7301b Compare June 12, 2025 16:17
@gooncreeper gooncreeper marked this pull request as draft June 12, 2025 21:31
@gooncreeper gooncreeper force-pushed the rename-circular-error branch from 5f7301b to e8f1be1 Compare June 16, 2025 01:33
@gooncreeper gooncreeper force-pushed the rename-circular-error branch from e8f1be1 to 539b31d Compare June 24, 2025 20:17
@gooncreeper gooncreeper changed the title rename: Add error.CircularLoop rename: Add and document errors caused by circular loops Jun 24, 2025
@gooncreeper gooncreeper marked this pull request as ready for review June 24, 2025 20:22
On POSIX systems, EINVAL is used to indicate circular loops;
additionally, corresponding with the use of the error in other
filesystem syscalls, it is used to indicate bad file names.

Based on empirical testing on Windows 10, SHARING_VIOLATION is
used to indicate a circular loop. On Windows 11, INVALID_PARAMATER
is used instead.
@gooncreeper gooncreeper force-pushed the rename-circular-error branch from 539b31d to 370c026 Compare June 24, 2025 20:27
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