fts: reduce two or more trailing slashes to just one, usually

[gnulib.git] / ChangeLog

diff --git a/ChangeLog b/ChangeLog
index 63f651f..0874c81 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,37 @@
+2012-09-04  Jim Meyering  <meyering@redhat.com>
+
+       fts: reduce two or more trailing spaces to just one, usually
+       * lib/fts.c (fts_open): Upon initialization, if a name ends in two
+       or more slashes, trim all but the final one.  But if a name consists
+       solely of two slashes, don't modify it.  If it consists solely of
+       three or more slashes, strip all but one.
+
+       This is part of the solution to a minor problem with rm:
+       it would print a bogus ELOOP diagnostic when failing to remove
+       the slash-decorated name of a symlink-to-directory:
+
+           $ mkdir d && ln -s d s && env rm -r s/
+           rm: cannot remove 's': Too many levels of symbolic links
+
+       With the change below and a trivial don't-trim-trailing-slashes
+       adjustment to remove.c, it does this:
+
+           $ env rm -r s/
+           rm: cannot remove 's/': Not a directory
+
+       Improved by: Eric Blake
+
+       fts: when there is no risk of overlap, use memcpy, not memmove
+       * lib/fts.c (fts_alloc): Fix unjustified memcopy: s/memmove/memcpy/
+
+2012-08-29  Paul Eggert  <eggert@cs.ucla.edu>
+
+       stdbool: be more compatible with mixed C/C++ compiles
+       * lib/stdbool.in.h (_Bool, true, false) [__cplusplus]:
+       Define to bool, true, false, respectively, as GCC's builtin
+       stdbool.h does.  Problem reported by Michael Goffioul in
+       <http://lists.gnu.org/archive/html/bug-gnulib/2012-08/msg00143.html>.
+
 2012-08-28  Jim Meyering  <meyering@redhat.com>
 
        revert last change: it was not needed