Thanks to visit codestin.com
Credit goes to github.com

Skip to content

gh-59616: Support os.chmod(follow_symlinks=True) and os.lchmod() on Windows #113049

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.

Sign up for GitHub

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

Merged
serhiy-storchaka merged 14 commits into from
Dec 14, 2023
Merged

gh-59616: Support os.chmod(follow_symlinks=True) and os.lchmod() on Windows #113049

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
0d117a1
gh-59616: Support os.chmod(follow_symlinks=True) and os.lchmod() on W…
serhiy-storchaka Dec 13, 2023
a69e8cc
Fix tests on Windows.
serhiy-storchaka Dec 13, 2023
167c00d
Fix clinic.
serhiy-storchaka Dec 13, 2023
ac3ed22
Fix clinic and Windows again.
serhiy-storchaka Dec 13, 2023
62a69c9
Update docs.
serhiy-storchaka Dec 13, 2023
aebcd68
Run all tests.
serhiy-storchaka Dec 13, 2023
1f47f6a
What's New.
serhiy-storchaka Dec 13, 2023
3f3e25a
Revert auditing changes.
serhiy-storchaka Dec 13, 2023
5f5f826
Fix test_shutil on Windows.
serhiy-storchaka Dec 13, 2023
64883ac
Lie to Sphinx.
serhiy-storchaka Dec 13, 2023
37725be
Apply suggestions from code review
serhiy-storchaka Dec 14, 2023
b1b2914
Merge branch 'main' into win-chmod-follow-symlinks
serhiy-storchaka Dec 14, 2023
fbf0f88
Merge branch 'main' into win-chmod-follow-symlinks
serhiy-storchaka Dec 14, 2023
17b0faf
Add more warnings about the default value.
serhiy-storchaka Dec 14, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Apply suggestions from code review 
Co-authored-by: Erlend E. Aasland <[email protected]>
  • Loading branch information
@serhiy-storchaka @erlend-aasland
serhiy-storchaka and erlend-aasland authored Dec 14, 2023
commit 37725be0abae3bd4f4e8c2972ef7187985a5ebc0
14 changes: 9 additions & 5 deletions Modules/posixmodule.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3323,10 +3323,12 @@ win32_lchmod(LPCWSTR path, int mode)
if (attr == INVALID_FILE_ATTRIBUTES) {
return 0;
}
if (mode & _S_IWRITE)
if (mode & _S_IWRITE) {
attr &= ~FILE_ATTRIBUTE_READONLY;
else
}
else {
attr |= FILE_ATTRIBUTE_READONLY;
}
return SetFileAttributesW(path, attr);
}
#endif
Expand Down Expand Up @@ -3404,14 +3406,16 @@ os_chmod_impl(PyObject *module, path_t *path, int mode, int dir_fd,
if (GetFileInformationByHandleEx(hfile, FileBasicInfo,
&info, sizeof(info)))
{
if (mode & _S_IWRITE)
if (mode & _S_IWRITE) {
info.FileAttributes &= ~FILE_ATTRIBUTE_READONLY;
else
}
else {
info.FileAttributes |= FILE_ATTRIBUTE_READONLY;
}
result = SetFileInformationByHandle(hfile, FileBasicInfo,
&info, sizeof(info));
}
CloseHandle(hfile);
(void)CloseHandle(hfile);
}
}
else {
Expand Down