chown --dereference did nothing when the owner/group of a
symlink matched the desired owner/group.  Reported by David Malone.
Also reported in 1999 as http://bugs.debian.org/39642.

(change_file_owner): When --dereference has
been specified, and when processing a symlink, stat it to get the
owner and group of the referent.

*** empty log message ***

In shred, check for errors from fdatasync more carefully.  If
fdatasync fails with errno==EINVAL, it means this implementation
does not support synchronized I/O for this file.  Do not report
this as an error, as (for example) AIX 5.2 fdatasync reports it
for raw disk devices.  Problem reported by Albert Chin in
<http://mail.gnu.org/archive/html/bug-gnu-utils/2004-05/msg00028.html>.

Check for write errors, though: the old code ignored them.
Improve error checking in a few other cases, too (e.g., close of a
directory).

Also, change several 'int' values to 'bool', so that the error
checking is a bit clearer.  Similarly, change unsigned values
to size_t where appropriate.

* src/shred.c: Include "dirname.h".
(datasync) [!HAVE_FDATASYNC]: Remove.
(dosync): New function.
(dopass): Use it.  Return 1 on write error, -1 on other error.
All callers changed.  Report write error if dosync does.
(do_wipefd, wipefd, wipename, wipefile): Return bool (true/false),
not int (0/-1).  All callers changed.  Return false if there's a
write error.
(incname): Return bool (true/false), not int (0/1).  Accept
size_t length, not unsigned.  All callers changed.  Do not
bother checking for non-digits; it can't happen.  Replace
recursion with iteration.
(wipename): Use dir_name, base_name, etc. instead of assuming
Unix file names.  Use size_t for length, not unsigned.
Report error if unlink or close fails.
(wipename, main): Use bool for booleans.

(names): Use only digits and uppercase letters, for greater
portability.

*** empty log message ***

*** empty log message ***

(TESTS): Add deref.

New test for the yesterday's change.

*** empty log message ***

.

(AD_pop_and_chdir): Eliminate an unnecessary call to AC_stack_top.

.

*** empty log message ***

Improve performance of `sort -m' on large files, at the cost of
making some contrived examples unsafe.  POSIX allows this
optimization.  Performance problem reported by Jonathan Baker in
<http://mail.gnu.org/archive/html/bug-coreutils/2004-05/msg00071.html>.

(first_same_file): Do not treat input pipes
differently from other files.

Document that "sort -m -o F" might write F before reading all the input.

*** empty log message ***

(sort invocation): Document that "sort -m -o F"
might write F before reading all the input.

*** empty log message ***

*** empty log message ***

Fix typo: s/supercede/supersede/.

*** empty log message ***

(unquote, make_printable_str): Use e.g. '\a' rather than '\007',
for portability to EBCDIC hosts.

(print_esc_char): Use e.g. '\a' rather than '\007',
for portability to EBCDIC hosts.

(print_ascii, dump_strings): Use e.g. '\a' rather than
'\007', for portability to EBCDIC hosts.

*** empty log message ***

Don't assume that "make -C" works; Solaris "make" doesn't have -C.

* src/Makefile.am (all_programs.list): New rule, copied from
man/Makefile.am and tests/Makefile.am, except that we use the
system tr rather than ./tr and we don't use tr -s.
* tests/Makefile.am (all_programs): Use it.
* man/Makefile.am (all_programs): Likewise.  Renamed from programs,
for consistency.  All uses changed.

Don't assume that "make -C" works; Solaris "make" doesn't have -C.

* src/Makefile.am (all_programs.list): New rule, copied from
man/Makefile.am and tests/Makefile.am, except that we use the
system tr rather than ./tr and we don't use tr -s.
* tests/Makefile.am (all_programs): Use it.

Don't assume that "make -C" works; Solaris "make" doesn't have -C.

(all_programs.list): New rule, copied from
man/Makefile.am and tests/Makefile.am, except that we use the
system tr rather than ./tr and we don't use tr -s.

*** empty log message ***

(AD_pop_and_chdir): Move lstat-`.' into if-block
where the result is used.  This avoids one unnecessary lstat call
per command line argument.

*** empty log message ***

(TESTS): Add unread3.

New test, for the above fix and today's lib/save-cwd.c improvement.

*** empty log message ***

(save_cwd): If opening `.' read-only fails, try write-only.

*** empty log message ***

*** empty log message ***

*** empty log message ***

(save_cwd) [HAVE_FCHDIR]: Don't fail just
because opening `.' failed.  Resort to using xgetcwd.

*** empty log message ***

Don't include "save-cwd.h".  It's no longer used.

.

*** empty log message ***

(TESTS): Add trap.

New file.  Test for bug fix of 2004-04-18.

*** empty log message ***

(AD_push): Don't use errno in diagnostic about `changed dev/ino'.

*** empty log message ***

Remove this generated file from CVS.

.

.

.

*** empty log message ***

(check-texinfo): grep -w IO

*** empty log message ***

(stat invocation): Change IO to I/O.

*** empty log message ***

(unquote): Use xcalloc rather than xmalloc and
a loop initializing the just-allocated memory to zero.

*** empty log message ***

*** empty log message ***

(TESTS): Add no-give-up.

Fix bug where "rm" gave up too easily, reported by Dan Jacobsen in
<http://mail.gnu.org/archive/html/bug-coreutils/2004-05/msg00013.html>.

(remove_entry): Check for errno values like ENOENT
that show the file cannot be directory, instead of for errno
values like EPERM that show the file might be a directory.  This
is necessary because, when a single unlink() call has multiple
reasons to fail, it can set errno to any of those reasons; it's
only the rare errno value like ENOENT that excludes all the other
possible reasons to fail even when the file is a directory.
(remove_cwd_entries): Don't attempt chdir if the file is known
to not be a directory.
(remove_dir): Use the same method that remove_cwd_entries uses
(for some reason they differed).  Don't assert that saved_errno
must be EPERM; it might be just about anything.

New file; check for the above fix.

Update from gnulib.
(obstack_finish): Rename local: s/value/__value/.

Update from gnulib.
(_): Define only if not already defined.

*** empty log message ***

(main): Use xnmalloc, rather than xmalloc.

(main): Use xnmalloc, rather than xmalloc.

(list_entries_users): Use xnmalloc, rather than xmalloc.

(do_wipefd): Use xnmalloc, rather than xmalloc.

(xgetgroups): Use variable name, rather than type name in computing buffer
size for xnmalloc.

(xgetgroups): Use xnmalloc, rather than xmalloc.
Don't add `1' to the buffer size (it was to protect against malloc
implementations that fail to allocate a buffer of size zero).
That is no longer necessary, since we use a malloc wrapper
on such systems.

(main): Use xnmalloc, rather than xmalloc.

(elide_tail_bytes_pipe): Use xnmalloc, rather than xmalloc.

(get_input_fstatus): Use xnmalloc, rather than xmalloc.

.

*** empty log message ***

(show_disk, show_point): If several filesystems are
mounted on the same mount point, prefer the last one, not the first.
Problem reported by Christian Jones in
<http://mail.gnu.org/archive/html/bug-coreutils/2004-04/msg00200.html>.
(show_disk): Remove unused statp arg.  Return bool, not int.
(show_point): Rewrite to avoid gotos.  Use the same algorithm
for lofs and dummies for each pass through the mount table,
rather than subtly different algorithms (which are probably
inadvertent).

*** empty log message ***

(lchown) [CHOWN_MODIFIES_SYMLINK]: Just call chown.

*** empty log message ***

*** empty log message ***

(EXTRA_DIST): Add m4/ChangeLog, now that we no longer have m4/Makefile*.

*** empty log message ***

.

Temporary work-around for the problem reported here:
http://sources.redhat.com/ml/automake/2004-05/msg00023.html
(FIXME: the number `23' may not be accurate)

(AM_GNU_GETTEXT): Don't require AM_INTL_SUBDIR.
(AM_INTL_SUBDIR): Comment out definition.

*** empty log message ***

Remove unused file.

Remove unused file.

Remove unused file.

Remove unused file.

Remove unused file.

Remove unused file.

Remove file.  This is now part of mktime.m4.

.

.

*** empty log message ***

(gl_MACROS): Move fchown-checking code to chown.m4.

(gl_PREREQ_CHOWN): Check for fcntl.h.
See if we need an fchown replacement.
(gl_FUNC_CHOWN_FOLLOWS_SYMLINK): New macro.
(gl_FUNC_CHOWN): Require gl_FUNC_CHOWN_FOLLOWS_SYMLINK,
and use the replacement function if we detect either defect.

(rpl_chown) [CHOWN_FAILS_TO_HONOR_ID_OF_NEGATIVE_ONE]:
Wrap old code with this conditional.
[CHOWN_MODIFIES_SYMLINK]: Try to work around a chown
function that does not dereference symlinks.

When chown or chgrp is modifying the referent of a symlink,
use the chown(2) function, if possible.

(change_file_owner): Don't hard-code the
open/fchown/close kludge here.  Use `chown' instead.
The chown function works just fine on conforming systems.
Other systems now go through the new chown wrapper that
resorts to the old kludge.