Re: Bug#669084: grep does not match anything when reading from stdin

[Aníbal Monsalve Salazar]

On Tue, Apr 17, 2012 at 01:45:43PM +0200, Jim Meyering wrote:
>Aníbal Monsalve Salazar wrote:
>...
>>Hello Jim,
>>
>>grep-2.11 introduced a regression related to the "-D skip" parameter
>>option.
>>
>>With grep-2.10 (or less than 2.10):
>>
>>$ echo foo | grep -D skip foo; echo $?
>>foo
>>0
>>
>>With grep-2.11:
>>
>>$ echo foo | grep -D skip foo; echo $?
>>1
>>
>>Tino suggests that the possible culprit is the code above.
>>
>>The Debian bug report is at http://bugs.debian.org/668585

Typo. The correct address is http://bugs.debian.org/669084

>Hi Aníbal
>
>Thanks for passing that along.
>Here's a lightly-tested patch:

Thanks. I'll build a new Debian package soon.

>From 415e4e69c8e8c0db288205b30ab2b6a337f62d38 Mon Sep 17 00:00:00 2001
>From: Jim Meyering <address@hidden>
>Date: Tue, 17 Apr 2012 13:37:10 +0200
>Subject: [PATCH] grep: --devices=ACTION (-D) no longer affects stdin
>MIME-Version: 1.0
>Content-Type: text/plain; charset=UTF-8
>Content-Transfer-Encoding: 8bit
>
>* doc/grep.texi (File and Directory Selection): Clarify this point,
>documenting the stdin exemption.
>* src/main.c (grepdesc): Ignore skip-related options when reading
>from standard input.
>* tests/skip-device: New file.  Test for the above.
>* tests/Makefile.am (TESTS): Add it.
>* NEWS (Bug fixes): Mention it, and add a few "[fixed in ...] notes.
>Reported by Sven Joachim in  http://bugs.debian.org/668585,

Reported by Tino Keitel in  http://bugs.debian.org/669084,

>and forwarded to bug-grep by Aníbal Monsalve Salazar.
>[...]