[debbugs-tracker] bug#17240: closed ([PATCH] grep: no match for the empty string included in multiple patterns)

[GNU bug Tracking System]

Your message dated Fri, 11 Apr 2014 13:25:18 -0700
with message-id <address@hidden>
and subject line Re: bug#17240: [PATCH] grep: no match for the empty string 
included in multiple patterns
has caused the debbugs.gnu.org bug report #17240,
regarding [PATCH] grep: no match for the empty string included in multiple 
patterns
to be marked as done.

(If you believe you have received this mail in error, please contact
address@hidden)


-- 
17240: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=17240
GNU Bug Tracking System
Contact address@hidden with problems

--- Begin Message --- Subject: [PATCH] grep: no match for the empty string included in multiple patterns Date: Fri, 11 Apr 2014 23:05:43 +0900

When the empty string is included in multiple, should match all lines.
However, `grep -F' and grep with regex isn't match.

$ printf 'foo\n\n' >pat
$ cat pat
foo

$ echo abc | env LANG=C grep -f pat
abc  <<== expected
$ echo abc | env LANG=C grep -Ff pat
$   <<== unexpected
$ printf '\(\)\\1x\n\n' >pat
$ cat pat
\(\)\1x

$ echo match | env LANG=C grep -f pat
$    <<== unexpected

Norihiro

patch.txt
Description: Text document

--- End Message ---

--- Begin Message --- Subject: Re: bug#17240: [PATCH] grep: no match for the empty string included in multiple patterns Date: Fri, 11 Apr 2014 13:25:18 -0700 User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.4.0 Thanks for catching that bug and for the fix. I found some ways to simplify it, and to fix a glitch with CRLF handling that I discovered while reviewing it, and installed your patch along with the attached two fixup patches.

0001-grep-cleanup-for-empty-string-fix.patch
Description: Text document

0002-grep-cleanup-for-HAS_DOS_FILE_CONTENTS-issue.patch
Description: Text document

--- End Message ---