Dump hacky overoptimisation in addir -- it doesn't actually get the case

[mmh] / ChangeLog

diff --git a/ChangeLog b/ChangeLog
index 1e9e31e..5f02b17 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,24 @@
+2008-08-04  Eric Gillespie  <epg@pretzelnet.org>
+
+       * uip/folder.c: Simplify dodir/addir/addfold.  Dump hacky
+       over-optimization in addir that tried to avoid readdir after all
+       child directories had been read; this was also trying to support
+       symlinks to directories, but would have been failing (because
+       nlink may have gone to 0 with symlinks to directories remaining)
+       had the lstat usage been correct (lstat doesn't fail for normal
+       directories; should have used S_ISLNK).
+
 2008-08-03  Peter Maydell  <pmaydell@chiark.greenend.org.uk>
 
+       * test/setup-test: use 'set -e' so we stop on compile failure.
+       Configure --enable-debug for convenience in debugging.
+
+       * uip/whatnowsbr.c: bug #23319: rework the way we handle the working
+       directory when invoking the user's shell, so that we don't have
+       issues when the cwd contains a space or shell special character.
+       * test/tests/whatnow/test-attach-detach, test/tests/whatnow/test-cd,
+       test/tests/whatnow/test-ls: test cases for this.
+
        * sbr/fmt_compile.c: add 'do { ... } while (0)' wrappers
        to various multi-statement macros to avoid nasty surprises
        if the macros are used in if() clauses.