diff options
author | Pádraig Brady <P@draigBrady.com> | 2020-01-30 20:46:40 -0500 |
---|---|---|
committer | Pádraig Brady <P@draigBrady.com> | 2020-02-04 19:11:37 +0000 |
commit | 1f443fe57291918ba245b2ffb7cdc07e64630af4 (patch) | |
tree | 6dcf3b66caca14a5d4aa68a4b7ee18134067d92a /THANKS.in | |
parent | 709b4d9d8dc2d456a2209b5ef3d1aaf0f1f45a48 (diff) | |
download | coreutils-1f443fe57291918ba245b2ffb7cdc07e64630af4.tar.gz |
rmdir: fix --ignore-fail-on-non-empty with permissions errors
Since v6.10-21-ged5c4e7 `rmdir --ignore-fail-on-non-empty`
had reversed the failure status for directories that failed
to be removed for permissions reasons. I.E. it would have
returned a failure status for such non empty dirs, and vice versa.
* src/rmdir.c (errno_may_be_non_empty): Rename from the
more confusing errno_may_be_empty(), and remove the EEXIST
case (specific to Solaris), which is moot here since
handled in errno_rmdir_non_empty().
(ignorable_failure): Fix the logic error so that
_non_ empty dirs are deemed to have ignorable failures.
(main): Fix clobbering of errno by is_empty_dir().
(remove_parents): Likewise.
* tests/rmdir/ignore.sh: Add a test case.
* THANKS.in: Add reporter who fixed the errno handling.
* NEWS: Mention the bug fix.
Fixes https://bugs.gnu.org/39364
Diffstat (limited to 'THANKS.in')
-rw-r--r-- | THANKS.in | 1 |
1 files changed, 1 insertions, 0 deletions
@@ -421,6 +421,7 @@ Matthew Arnison maffew@cat.org.au Matthew Braun matthew@ans.net Matthew Clarke Matthew_Clarke@mindlink.bc.ca Matthew M. Boedicker matthewm@boedicker.org +Matthew Pfeiffer spferical@gmail.com Matthew S. Levine mslevine@theory.lcs.mit.edu Matthew Smith matts@bluesguitar.org Matthew Swift swift@alum.mit.edu |