* THANKS: Add Andreas Frische.

* NEWS: Mention today's ls --color fix.
* tests/ls-2/tests (sl-dangle): Add a test for today's fix.

diff --git a/ChangeLog b/ChangeLog
index b90878cf9803d9713b6d21d4d22e1ed5a188e65d..c3a73ffa70ea8456d1ea1cbfa76dcd52d2ad9893 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2007-04-24  Jim Meyering  <jim@meyering.net>
+
+       * THANKS: Add Andreas Frische.
+       * NEWS: Mention today's ls --color fix.
+       * tests/ls-2/tests (sl-dangle): Add a test for today's fix.
+
 2007-04-24  Eric Blake  <ebb9@byu.net>
 
        ls --color once again colors dangling symlinks correctly

diff --git a/NEWS b/NEWS
index c4678e7dd8bb7d6a5ca12f30068d5b390b90985d..0542cd98746248e72d10ee0c4133b454f60520cd 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -11,6 +11,11 @@ GNU coreutils NEWS                                    -*- outline -*-
   ls -x DIR would sometimes output the wrong string in place of the
   first entry.  [introduced in coreutils-6.8]
 
+  ls --color would mistakenly color a dangling symlink as if it were
+  a regular symlink.  This would happen only when the dangling symlink
+  was not a command-line argument and in a directory with d_type support.
+  [introduced in coreutils-6.0]
+
   ls --color, (with a custom LS_COLORS envvar value including the
   ln=target attribute) would mistakenly output the string "target"
   before the name of each symlink.  [introduced in coreutils-6.0]

diff --git a/THANKS b/THANKS
index 63fcc39f8d24e7c8388d621671a6145393aaaa1c..2033c38453e6ac0e5874817e9e903e9fdd19d3a8 100644 (file)
--- a/THANKS
+++ b/THANKS
@@ -26,6 +26,7 @@ Alexey Vyskubov                     alexey@pippuri.mawhrin.net
 Alfred M. Szmidt                    ams@kemisten.nu
 Andi Kleen                          freitag@alancoxonachip.com
 Andre Novaes Cunha                  Andre.Cunha@br.global-one.net
+Andreas Frische                     andreasfrische@gmail.com
 Andreas Gruenbacher                 ag@bestbits.at
 Andreas Jaeger                      jaeger@gnu.org
 Andreas Luik                        luik@isa.de

diff --git a/tests/ls-2/tests b/tests/ls-2/tests
index 3f2fc1df7b94d69152969e7a403296f085794e8c..9bf40577d38117fd6c7a7d64490db3146c61be1f 100755 (executable)
--- a/tests/ls-2/tests
+++ b/tests/ls-2/tests
@@ -130,6 +130,19 @@ my @Tests =
      ['sl-target', '--color=always d',
       {OUT => "\e[0m\e[01;34mX\e[0m\n\e[m"}, $target, $target2],
 
+     # Test for another bug fixed in coreutils-6.10.
+     # This one bites only for a system/file system with d_type support.
+     ['sl-dangle', '--color=always d',
+      {OUT => "\e[0m\e[40;31;01mX\e[0m\n\e[m"},
+      {PRE => sub {
+               mkdir 'd',0755 or die "d: $!\n";
+               symlink 'non-existent', 'd/X' or die "d/X: $!\n";
+               $ENV{LS_COLORS}='or=40;31;01'
+             }},
+      {POST => sub {unlink 'd/X' or die "d/X: $!\n";
+                   rmdir 'd' or die "d: $!\n" }},
+     ],
+
      # Test for a bug that was introduced in coreutils-4.5.4; fixed in 4.5.5.
      # To demonstrate it, the file in question (with executable bit set)
      # must not be a command line argument.