diff options
| author | Rob Day <robertkday@gmail.com> | 2012-08-22 23:04:19 +0100 |
|---|---|---|
| committer | Jim Meyering <meyering@redhat.com> | 2012-08-23 20:53:04 +0200 |
| commit | dd22da8e9539cc88193987b6997769ae4ede2b15 (patch) | |
| tree | 630d3469f4d7710c3252682a51f495eba1846924 /NEWS | |
| parent | fadf9c525787f063e6fe84bdd5566e53a622c1ce (diff) | |
| download | coreutils-dd22da8e9539cc88193987b6997769ae4ede2b15.tar.xz | |
rm: fix the new --dir (-d) option to work with -i
* src/remove.c (prompt): Hoist the computation of is_empty, since we'll
need it slightly earlier.
Before, this function would arrange to fail with EISDIR when processing
a directory without --recursive (-r). Adjust the condition to exempt
an empty directory when --dir has been specified.
Improve comments.
* tests/rm/d-3: New file, to ensure that rm -d -i dir works.
* tests/Makefile.am (TESTS): Add it.
* NEWS (Bug fixes): Mention it.
* THANKS.in: Update.
Reported by Michael Price in http://bugs.gnu.org/12260
Diffstat (limited to 'NEWS')
| -rw-r--r-- | NEWS | 4 |
1 files changed, 4 insertions, 0 deletions
@@ -9,6 +9,10 @@ GNU coreutils NEWS -*- outline -*- it detects this precise type of cycle, diagnoses it as such and eventually exits nonzero. + rm -i -d now prompts the user then removes an empty directory, rather + than ignoring the -d option and failing with an 'Is a directory' error. + [bug introduced in coreutils-8.19, with the addition of --dir (-d)] + * Noteworthy changes in release 8.19 (2012-08-20) [stable] |