coreutils/src/remove.c, branch v6.7

* src/remove.c (fd_to_subdirp): Remove unused parameter, ds.

2006-11-26T17:47:54Z

Update callers.

Port parts of the code to C89 to minimize the need for c99-to-c89.diff,

2006-11-26T16:40:38Z

while trying to retain the readability of C99 as much as possible. * src/remove.c (close_preserve_errno): Remove. (fd_to_subdirp): Rewrite to avoid the need for decl after statement. Signed-off-by: Paul Eggert

* src/remove.c (rm): Move cycle_check_init call into callee...

2006-11-26T16:40:37Z

Use an else clause in place of a "continue" statement. Signed-off-by: Paul Eggert

* src/remove.c (AD_ensure_initialized): New function.

2006-11-26T16:40:16Z

after statement. Signed-off-by: Paul Eggert

* src/remove.c (AD_pop_and_chdir): Return prev_dir rather than storing through

2006-11-26T16:39:53Z

a pointer argument. All uses changed. Signed-off-by: Paul Eggert

* src/remove.c (cache_stat_init): Return its argument, for convenience.

2006-11-26T16:39:27Z

Update the caller in remove_dir. Signed-off-by: Paul Eggert

* src/remove.c (rm_1): Remove decl of local, fd_cwd.

2006-11-26T16:38:09Z

Replace each of two uses with literal AT_FDCWD. Signed-off-by: Paul Eggert

new feature: rm accepts new option: --one-file-system

2006-10-24T22:01:33Z

Suggested by Steve McIntyre in . * src/remove.h (struct rm_options) [one_file_system]: New member. * src/rm.c (rm_option_init): Initialize it. (usage): Document the option. * src/mv.c (rm_option_init): Likewise. * src/remove.c (remove_dir): With --one-file-system and --recursive, for each directory command line argument, do not affect a file system different from that of the starting directory. And give a diagnostic. * src/rm.c (ONE_FILE_SYSTEM): New enum. (main): Handle new option. * tests/rm/one-file-system: Test the above. * tests/rm/Makefile.am (TESTS): Add one-file-system. * tests/Makefile.am (check-root): Add the rm/one-file-system test to the list. (EXTRA_DIST): Add other-fs-tmpdir. * tests/mv/setup: Removed. Renamed to... * tests/other-fs-tmpdir: ...this new file. * tests/mv/Makefile.am (EXTRA_DIST): Remove setup. * tests/mv/acl: Reflect renaming: use ../other-fs-tmpdir. * tests/mv/backup-is-src: Likewise. * tests/mv/hard-link-1: Likewise. * tests/mv/leak-fd: Likewise. * tests/mv/mv-special-1: Likewise. * tests/mv/part-fail: Likewise. * tests/mv/part-hardlink: Likewise. * tests/mv/part-rename: Likewise. * tests/mv/part-symlink: Likewise. * tests/mv/partition-perm: Likewise. * tests/mv/to-symlink: Likewise. * tests/mv/into-self-2: Likewise. [doc/ChangeLog] * coreutils.texi (rm invocation): Describe --one-file-system.

* src/dd.c (flags): noatime and nofollow now depend on

2006-10-05T21:39:12Z

HAVE_WORKING_O_NOATIME and HAVE_WORKING_O_NOFOLLOW, too. (usage): Output info about noatime and nofollow only if they are known to work. * src/remove.c (AD_push): Inspect HAVE_WORKING_O_NOFOLLOW rather than O_NOFOLLOW, when testing whether it's possible to avoid a race condition reliably.

* src/remove.c (nonexistent_file_errno): Remove ENAMETOOLONG.

2006-10-03T22:06:31Z

Paul Eggert pointed out that the specified file may exist, in spite of such an errno value. * tests/rm/Makefile.am (TESTS): Remove ignore-name-too-long. * tests/rm/ignore-name-too-long: Remove file.