mirror of
https://github.com/coreutils/coreutils.git
synced 2024-11-24 10:23:31 +08:00
rm: fix --one-file-system regression due to fts conversion
* src/remove.c (rm_fts): Fix incorrect comparison of device and inode numbers. * tests/rm/one-file-system2: Add a separate test so that it can be run as a normal user (It doesn't need to mount). * tests/Makefile.am: Reference it. * NEWS: Mention the fix. Reported by Jan Larres.
This commit is contained in:
parent
bfcfc0ce72
commit
d64c186d8e
7
NEWS
7
NEWS
@ -2,6 +2,13 @@ GNU coreutils NEWS -*- outline -*-
|
|||||||
|
|
||||||
* Noteworthy changes in release ?.? (????-??-??) [?]
|
* Noteworthy changes in release ?.? (????-??-??) [?]
|
||||||
|
|
||||||
|
** Bug fixes
|
||||||
|
|
||||||
|
rm -r --one-file-system works once again.
|
||||||
|
The rewrite to make rm use fts introduced a regression whereby
|
||||||
|
a commmand of the above form would fail for all subdirectories.
|
||||||
|
[bug introduced in coreutils-8.0]
|
||||||
|
|
||||||
|
|
||||||
* Noteworthy changes in release 8.2 (2009-12-11) [stable]
|
* Noteworthy changes in release 8.2 (2009-12-11) [stable]
|
||||||
|
|
||||||
|
@ -548,7 +548,7 @@ rm_fts (FTS *fts, FTSENT *ent, struct rm_options const *x)
|
|||||||
if (ent->fts_info == FTS_DP
|
if (ent->fts_info == FTS_DP
|
||||||
&& x->one_file_system
|
&& x->one_file_system
|
||||||
&& FTS_ROOTLEVEL < ent->fts_level
|
&& FTS_ROOTLEVEL < ent->fts_level
|
||||||
&& ent->fts_statp->st_ino != fts->fts_dev)
|
&& ent->fts_statp->st_dev != fts->fts_dev)
|
||||||
{
|
{
|
||||||
mark_ancestor_dirs (ent);
|
mark_ancestor_dirs (ent);
|
||||||
error (0, 0, _("skipping %s, since it's on a different device"),
|
error (0, 0, _("skipping %s, since it's on a different device"),
|
||||||
|
@ -107,6 +107,7 @@ TESTS = \
|
|||||||
rm/interactive-always \
|
rm/interactive-always \
|
||||||
rm/interactive-once \
|
rm/interactive-once \
|
||||||
rm/ir-1 \
|
rm/ir-1 \
|
||||||
|
rm/one-file-system2 \
|
||||||
rm/r-1 \
|
rm/r-1 \
|
||||||
rm/r-2 \
|
rm/r-2 \
|
||||||
rm/r-3 \
|
rm/r-3 \
|
||||||
|
31
tests/rm/one-file-system2
Executable file
31
tests/rm/one-file-system2
Executable file
@ -0,0 +1,31 @@
|
|||||||
|
#!/bin/sh
|
||||||
|
# Verify --one-file-system does delete within a file system
|
||||||
|
|
||||||
|
# Copyright (C) 2009 Free Software Foundation, Inc.
|
||||||
|
|
||||||
|
# This program is free software: you can redistribute it and/or modify
|
||||||
|
# it under the terms of the GNU General Public License as published by
|
||||||
|
# the Free Software Foundation, either version 3 of the License, or
|
||||||
|
# (at your option) any later version.
|
||||||
|
|
||||||
|
# This program is distributed in the hope that it will be useful,
|
||||||
|
# but WITHOUT ANY WARRANTY; without even the implied warranty of
|
||||||
|
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
||||||
|
# GNU General Public License for more details.
|
||||||
|
|
||||||
|
# You should have received a copy of the GNU General Public License
|
||||||
|
# along with this program. If not, see <http://www.gnu.org/licenses/>.
|
||||||
|
|
||||||
|
if test "$VERBOSE" = yes; then
|
||||||
|
set -x
|
||||||
|
rm --version
|
||||||
|
fi
|
||||||
|
|
||||||
|
. $srcdir/test-lib.sh
|
||||||
|
|
||||||
|
mkdir -p a/b
|
||||||
|
|
||||||
|
rm --one-file-system -rf a || fail=1
|
||||||
|
test -d a && fail=1
|
||||||
|
|
||||||
|
Exit $fail
|
Loading…
Reference in New Issue
Block a user