summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--NEWS4
-rw-r--r--doc/coreutils.texi17
-rw-r--r--src/tee.c9
-rw-r--r--tests/local.mk1
-rwxr-xr-xtests/misc/tee-dash.sh24
-rwxr-xr-xtests/misc/tee.sh25
6 files changed, 31 insertions, 49 deletions
diff --git a/NEWS b/NEWS
index b6795aacf..859735743 100644
--- a/NEWS
+++ b/NEWS
@@ -73,6 +73,10 @@ GNU coreutils NEWS -*- outline -*-
tee will exit early if there are no more writable outputs.
+ tee does not treat the file operand '-' as meaning standard output any longer,
+ for better conformance to POSIX. This feature was added in coreutils-5.3.0.
+
+
** Improvements
cp,install,mv will convert smaller runs of NULs in the input to holes,
diff --git a/doc/coreutils.texi b/doc/coreutils.texi
index bb652aca5..8d050c2e8 100644
--- a/doc/coreutils.texi
+++ b/doc/coreutils.texi
@@ -752,11 +752,10 @@ operands even if they begin with @samp{-}. For example, @samp{sort --
@cindex standard input
@cindex standard output
A single @samp{-} operand is not really an option, though it looks like one. It
-stands for standard input, or for standard output if that is clear from
-the context. For example, @samp{sort -} reads from standard input,
-and is equivalent to plain @samp{sort}, and @samp{tee -} writes an
-extra copy of its input to standard output. Unless otherwise
-specified, @samp{-} can appear as any operand that requires a file
+stands for a file operand, and some tools treat it as standard input, or as
+standard output if that is clear from the context. For example, @samp{sort -}
+reads from standard input, and is equivalent to plain @samp{sort}. Unless
+otherwise specified, a @samp{-} can appear as any operand that requires a file
name.
@menu
@@ -13180,9 +13179,11 @@ If a file being written to does not already exist, it is created. If a
file being written to already exists, the data it previously contained
is overwritten unless the @option{-a} option is used.
-A @var{file} of @samp{-} causes @command{tee} to send another copy of
-input to standard output, but this is typically not that useful as the
-copies are interleaved.
+In previous versions of GNU coreutils (v5.3.0 - v8.23), a @var{file} of @samp{-}
+caused @command{tee} to send another copy of input to standard output.
+However, as the interleaved output was not very useful, @command{tee} now
+conforms to POSIX which explicitly mandates it to treat @samp{-} as a file
+with such name.
The program accepts the following options. Also see @ref{Common options}.
diff --git a/src/tee.c b/src/tee.c
index bfe1b6900..feb402628 100644
--- a/src/tee.c
+++ b/src/tee.c
@@ -68,10 +68,6 @@ Copy standard input to each FILE, and also to standard output.\n\
"), stdout);
fputs (HELP_OPTION_DESCRIPTION, stdout);
fputs (VERSION_OPTION_DESCRIPTION, stdout);
- fputs (_("\
-\n\
-If a FILE is -, copy again to standard output.\n\
-"), stdout);
emit_ancillary_info (PROGRAM_NAME);
}
exit (status);
@@ -169,9 +165,8 @@ tee_files (int nfiles, const char **files)
for (i = 1; i <= nfiles; i++)
{
- descriptors[i] = (STREQ (files[i], "-")
- ? stdout
- : fopen (files[i], mode_string));
+ /* Do not treat "-" specially - as mandated by POSIX. */
+ descriptors[i] = fopen (files[i], mode_string);
if (descriptors[i] == NULL)
{
error (0, errno, "%s", files[i]);
diff --git a/tests/local.mk b/tests/local.mk
index 53c7c83e0..9a5208082 100644
--- a/tests/local.mk
+++ b/tests/local.mk
@@ -382,7 +382,6 @@ all_tests = \
tests/misc/tac-2-nonseekable.sh \
tests/misc/tail.pl \
tests/misc/tee.sh \
- tests/misc/tee-dash.sh \
tests/misc/test-diag.pl \
tests/misc/timeout.sh \
tests/misc/timeout-blocked.pl \
diff --git a/tests/misc/tee-dash.sh b/tests/misc/tee-dash.sh
deleted file mode 100755
index 1063211cf..000000000
--- a/tests/misc/tee-dash.sh
+++ /dev/null
@@ -1,24 +0,0 @@
-#!/bin/sh
-# test for "tee -".
-
-# Copyright (C) 2005-2015 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 3 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, see <http://www.gnu.org/licenses/>.
-
-. "${srcdir=.}/tests/init.sh"; path_prepend_ ./src
-print_ver_ tee
-
-tee - </dev/null || fail=1
-
-Exit $fail
diff --git a/tests/misc/tee.sh b/tests/misc/tee.sh
index 5f2eeda7b..ecca6562b 100755
--- a/tests/misc/tee.sh
+++ b/tests/misc/tee.sh
@@ -20,17 +20,24 @@
print_ver_ tee
echo line >sample || framework_failure_
-nums=$(seq 9) || framework_failure_
-
-for n in 0 $nums; do
- files=$(seq $n)
- rm -f $files
- tee $files <sample >out || fail=1
- for f in out $files; do
- compare sample $f || fail=1
- done
+
+# POSIX says: "Processing of at least 13 file operands shall be supported."
+for n in 0 1 2 12 13; do
+ files=$(seq $n)
+ rm -f $files
+ tee $files <sample >out || fail=1
+ for f in out $files; do
+ compare sample $f || fail=1
+ done
done
+# Ensure tee treats '-' as the name of a file, as mandated by POSIX.
+# Between v5.3.0 and v8.23, a '-' argument caused tee to send another
+# copy of input to standard output.
+tee - <sample >out 2>err || fail=1
+compare sample ./- || fail=1
+compare sample out || fail=1
+compare /dev/null err || fail
# Ensure tee exits early if no more writable outputs
if test -w /dev/full && test -c /dev/full; then