From a3b65a63a0ecbd617f59572add0109e5e5944acf Mon Sep 17 00:00:00 2001 From: Paul Eggert Date: Mon, 2 Apr 2007 18:27:34 +0200 Subject: * src/ls.c (print_horizontal): Fix bug reported by Mike Frysinger: ls -x DIR would sometimes output the wrong string in place of the first entry. * NEWS: Mention the bug fix. * tests/ls/x-option: New file. * tests/ls/Makefile.am (TESTS): Add x-option. --- tests/ls/Makefile.am | 2 +- tests/ls/x-option | 59 ++++++++++++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 60 insertions(+), 1 deletion(-) create mode 100755 tests/ls/x-option (limited to 'tests') diff --git a/tests/ls/Makefile.am b/tests/ls/Makefile.am index 461ba0756..2b6fb9044 100644 --- a/tests/ls/Makefile.am +++ b/tests/ls/Makefile.am @@ -24,7 +24,7 @@ TESTS = \ stat-dtype \ inode dangle file-type recursive dired infloop \ rt-1 time-1 symlink-slash follow-slink no-arg m-option \ - stat-vs-dirent + stat-vs-dirent x-option EXTRA_DIST = $(TESTS) TESTS_ENVIRONMENT = \ diff --git a/tests/ls/x-option b/tests/ls/x-option new file mode 100755 index 000000000..44556dfca --- /dev/null +++ b/tests/ls/x-option @@ -0,0 +1,59 @@ +#!/bin/sh +# Exercise the -x option. + +# Copyright (C) 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 +# the Free Software Foundation; either version 2 of the License, or +# (at your option) any later version. + +# This program is distributed in the hope that it will be useful, +# but WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +# GNU General Public License for more details. + +# You should have received a copy of the GNU General Public License +# along with this program; if not, write to the Free Software +# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA +# 02110-1301, USA. + +if test "$VERBOSE" = yes; then + set -x + ls --version +fi + +. $srcdir/../envvar-check +. $srcdir/../lang-default + +pwd=`pwd` +t0=`echo "$0"|sed 's,.*/,,'`.tmp; tmp=$t0/$$ +trap 'status=$?; cd "$pwd" && chmod -R u+rwx $t0 && rm -rf $t0 && exit $status' 0 +trap '(exit $?); exit $?' 1 2 13 15 + +framework_failure=0 +mkdir -p $tmp || framework_failure=1 +cd $tmp || framework_failure=1 +mkdir subdir || framework_failure=1 +touch subdir/b || framework_failure=1 +touch subdir/a || framework_failure=1 + +if test $framework_failure = 1; then + echo "$0: failure in testing framework" 1>&2 + (exit 1); exit 1 +fi + +fail=0 + +# Coreutils 6.8 and 6.9 would output this in the wrong order. +ls -x subdir > out || fail=1 +ls -rx subdir >> out || fail=1 +cat <<\EOF > exp || fail=1 +a b +b a +EOF + +cmp out exp || fail=1 +test $fail = 1 && diff out exp 2> /dev/null + +(exit $fail); exit $fail -- cgit v1.2.3-54-g00ecf