From 4d484fe09feb198a8129fea29b4db50891988f62 Mon Sep 17 00:00:00 2001 From: Jim Meyering Date: Sat, 7 Apr 2007 20:46:27 +0200 Subject: Add a test for, and document, today's fix. * NEWS: Mention today's ls --color fix. * tests/ls-2/tests (sl-target): Add a test for today's fix. * THANKS: Add Kirk Kelsey. --- ChangeLog | 7 +++++++ NEWS | 4 ++++ THANKS | 1 + tests/ls-2/tests | 14 ++++++++++++-- 4 files changed, 24 insertions(+), 2 deletions(-) diff --git a/ChangeLog b/ChangeLog index f95dcfd86..a40c398d5 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,10 @@ +2007-04-07 Jim Meyering + + Add a test for, and document, today's fix. + * NEWS: Mention today's ls --color fix. + * tests/ls-2/tests (sl-target): Add a test for today's fix. + * THANKS: Add Kirk Kelsey. + 2007-04-07 Eric Blake Fix a bug in how the LS_COLORS ln=target attribute is handled. diff --git a/NEWS b/NEWS index 6dee2cb84..c1e4bf9fb 100644 --- a/NEWS +++ b/NEWS @@ -11,6 +11,10 @@ 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, (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] + * Noteworthy changes in release 6.9 (2007-03-22) [stable] diff --git a/THANKS b/THANKS index ad05b1350..b353b8f97 100644 --- a/THANKS +++ b/THANKS @@ -274,6 +274,7 @@ Keith Owens kaos@audio.apana.org.au Keith Thompson kst@cts.com Ken Pizzini kenp@halcyon.com Kevin Mudrick kmudrick@healthmarketscience.com +Kirk Kelsey kirk.kelsey@0x4b.net Kristin E Thomas kristint@us.ibm.com Kjetil Torgrim Homme kjetilho@ifi.uio.no Kristoffer Rose kris@diku.dk diff --git a/tests/ls-2/tests b/tests/ls-2/tests index d0257a6a3..3f2fc1df7 100755 --- a/tests/ls-2/tests +++ b/tests/ls-2/tests @@ -1,8 +1,7 @@ #!/bin/sh # -*- perl -*- -# Copyright (C) 1998, 2000, 2001, 2002, 2003, 2004, 2005, 2006 Free -# Software Foundation, Inc. +# Copyright (C) 1998, 2000-2007 Free Software Foundation, Inc. # This program is free software; you can redistribute it and/or modify # it under the terms of the GNU General Public License as published by @@ -58,6 +57,13 @@ my $mkdir2 = {PRE => sub {mkdir 'd',0755 or die "d: $!\n"; my $rmdir2 = {POST => sub {rmdir 'd/e' or die "d/e: $!\n"; rmdir 'd' or die "d: $!\n" }}; +my $target = {PRE => sub { + mkdir 'd',0755 or die "d: $!\n"; + symlink '.', 'd/X' or die "d/X: $!\n"; + $ENV{LS_COLORS}='ln=target' + }}; +my $target2 = {POST => sub {unlink 'd/X' or die "d/X: $!\n"; + rmdir 'd' or die "d: $!\n" }}; my $slink_d = {PRE => sub {symlink '/', 'd' or die "d: $!\n"; $ENV{LS_COLORS}='ln=01;36:di=01;34:or=40;31;01' }}; @@ -120,6 +126,10 @@ my @Tests = {OUT => "\e[0m\e[01;36md\e[0m\@\n\e[m"}, $slink_d, $unlink_d], + # Test for a bug fixed in coreutils-6.10. + ['sl-target', '--color=always d', + {OUT => "\e[0m\e[01;34mX\e[0m\n\e[m"}, $target, $target2], + # 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. -- cgit v1.2.3-54-g00ecf