chmod: output permission denied error messages for inaccesible directories/files as regular user#9802
chmod: output permission denied error messages for inaccesible directories/files as regular user#9802asder8215 wants to merge 6 commits intouutils:mainfrom
Conversation
|
GNU testsuite comparison: |
|
I'm not sure if any of the errors on CICD/Build + L10n are the result of the changes I made to this code (it shouldn't affect anything because I just change the implementation of the code from using As for the WSL2 error, I ran the |
|
sorry, it needs to be rebased |
e77c343 to
1c0f4c0
Compare
|
@sylvestre, I just resetted the branch to the current updates that coreutils' main has and reapplied my changes within |
b253f05 to
07746a4
Compare
|
GNU testsuite comparison: |
|
@sylvestre I think this PR is ready for review. I'm pretty certain that the GNU test failures for Also, I noticed another PR #9860 solving the same issue I am doing here. I'm open to either approach (using |
This PR modifies checking if a file exists from using
.exists()to.try_exists()(handling it withmatchstatements thanifconditionals). This allows us to check for errors likePermissionDeniedand output the error message accordingly for inaccesible directories/files as regular user. This fixes #9789.