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 <[email protected]> >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. >[...]
