tail: let -f --retry wait for inaccessible files
commitd461bfd2743547360793acd2118d4c08ad7b84a8
authorBernhard Voelker <mail@bernhard-voelker.de>
Sat, 20 Apr 2013 14:33:06 +0000 (20 16:33 +0200)
committerBernhard Voelker <mail@bernhard-voelker.de>
Sat, 20 Apr 2013 14:33:06 +0000 (20 16:33 +0200)
tree6e81dc004be45e9dea0c7b469b7d98c9f0277950
parent1dd8a33169b69716c4a4b92dbe1bd99336d92b23
tail: let -f --retry wait for inaccessible files

The --retry option is indeed useful for both following modes
by name and by file descriptor.  The difference is that in the
latter case, it is effective only during the initial open.

As a regression of the implementation of the inotify support,
tail -f --retry would immediately exit if the given file is
inaccessible.

* src/tail.c (usage): Change the description of the --retry option:
remove the note that this option would mainly be useful when
following by name.
(main): Change diagnosing dubios uses of --retry option:
when the --retry option is used without following, then issue
a warning that this option is ignored; when it is used together
with --follow=descriptor, then issue a warning that it is only
effective for the initial open.
Disable inotify also in the case when the initial open in tail_file()
failed (which is the actual bug fix).
* init.cfg (retry_delay_): Pass excess arguments to the test function.
* tests/tail-2/retry.sh: Add new tests.
* tests/local.mk (all_tests): Mention it.
* doc/coreutils.texi (tail invocation): Enhance the documentation
of the --retry option.  Clarify the difference in tail's behavior
regarding the --retry option when combined with the following modes
name versus descriptor.
* NEWS (Bug fixes): Mention the fix.

Reported by Noel Morrison in:
http://lists.gnu.org/archive/html/coreutils/2013-04/msg00003.html
NEWS
doc/coreutils.texi
init.cfg
src/tail.c
tests/local.mk
tests/tail-2/retry.sh [new file with mode: 0644]